Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EXP][CMD-BUFFER] Add read and write function definitions #644

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

mfrancepillois
Copy link
Contributor

Add new command-buffer functions for appending read and write commands for one-dimensional and rectangular buffers

scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
@mfrancepillois mfrancepillois force-pushed the maxime/command-buffer-read-write branch from b460bbc to fb3ab63 Compare June 22, 2023 10:55
Copy link
Contributor

@Bensuo Bensuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just some minor comments

scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
scripts/core/exp-command-buffer.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@kbenzie kbenzie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does scripts/core/EXP-COMMAND-BUFFER.rst need updated?

Copy link
Contributor

@Bensuo Bensuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one minor comment!

scripts/core/EXP-COMMAND-BUFFER.rst Outdated Show resolved Hide resolved
@mfrancepillois mfrancepillois force-pushed the maxime/command-buffer-read-write branch from 7f57818 to 85fabe8 Compare June 22, 2023 16:05
Copy link
Contributor

@veselypeta veselypeta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT - just blocking this to fix the registry.

scripts/core/registry.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@kbenzie kbenzie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One last change, otherwise looking good!

scripts/core/registry.yml Outdated Show resolved Hide resolved
@mfrancepillois mfrancepillois force-pushed the maxime/command-buffer-read-write branch from 85fabe8 to 9019018 Compare June 23, 2023 11:01
@veselypeta veselypeta merged commit c7b0caf into oneapi-src:main Jun 23, 2023
EwanC added a commit to reble/llvm that referenced this pull request Jun 27, 2023
Pulls in changes from #238
which requires bumping the UR commit to
oneapi-src/unified-runtime#644

Co-authored-by: Maxime France-Pillois <maxime.francepillois@codeplay.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants