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

[SYCL][Graph] Update doc for UR PR moving reset commands to a dedicated cmd-list #12770

Merged
merged 5 commits into from
Mar 14, 2024

Conversation

mfrancepillois
Copy link
Contributor

Update the design doc.
Update the UR tag.

Copy link
Contributor

@aelovikov-intel aelovikov-intel left a comment

Choose a reason for hiding this comment

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

I don't see what needs to be reviewed by the SYCL RT, probably imperfect CODEOWNERS.

@kbenzie
Copy link
Contributor

kbenzie commented Feb 21, 2024

I don't see what needs to be reviewed by the SYCL RT, probably imperfect CODEOWNERS.

It's the design docs setup https://github.com/intel/llvm/blob/sycl/.github/CODEOWNERS#L33

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.

oneapi-src/unified-runtime#1365 has been merged, please pull in the latest sycl branch changes, resolve the conflict and update the UR repo/tag as suggested.

sycl/plugins/unified_runtime/CMakeLists.txt Outdated Show resolved Hide resolved
EwanC and others added 2 commits March 14, 2024 10:25
@EwanC
Copy link
Contributor

EwanC commented Mar 14, 2024

@intel/llvm-gatekeepers This is good to merge

@steffenlarsen steffenlarsen merged commit 8850a97 into intel:sycl Mar 14, 2024
12 checks passed
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.

5 participants