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] Re-apply task sequence changes #14750

Merged
merged 17 commits into from
Jul 26, 2024
Merged

Conversation

aejjehint
Copy link
Contributor

This change re-apply the task sequence changes from PR #12453 that were reverted in #14359 now that the spec has been approved and the back-end implementation is ready. The spec is also moved from proposed to experimental.

@aejjehint aejjehint requested a review from a team as a code owner July 24, 2024 17:52
Copy link
Contributor

@mdtoguchi mdtoguchi left a comment

Choose a reason for hiding this comment

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

Looks OK for Driver

@smanna12
Copy link
Contributor

@steffenlarsen, @AlexeySachkov, @GarveyJoe, could you please review this PR? Thank you

@steffenlarsen steffenlarsen changed the title re-apply task sequence changes [SYCL] Re-apply task sequence changes Jul 26, 2024
Copy link
Contributor

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

Overall I think it looks good!

aejjehint and others added 4 commits July 26, 2024 08:54
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
aejjehint and others added 3 commits July 26, 2024 08:55
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
…t.cpp

Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
@aejjehint
Copy link
Contributor Author

@intel/llvm-gatekeepers PR is ready to merge

@againull againull merged commit 7e6db77 into intel:sycl Jul 26, 2024
15 checks passed
@againull
Copy link
Contributor

Notified @aejjehint about post-commit failures: https://github.com/intel/llvm/actions/runs/10117762715/job/27983612239

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.

7 participants