-
Notifications
You must be signed in to change notification settings - Fork 734
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] Support for Prefetch and memory advise #11474
[SYCL][Graph] Support for Prefetch and memory advise #11474
Conversation
Adds support for `prefetch` and `advise` memory hints. Adds e2e tests that verify that backend functions are called as they should be.
… enqueuing commands to a command-buffer.
…eble/llvm into command-buffer-prefetch-advise-support
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
ping @intel/llvm-reviewers-runtime for a review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Runtime changes LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Native CPU lgtm
tagging @intel/llvm-gatekeepers for merging |
Fixes post-commit CI failure seen here https://github.com/intel/llvm/actions/runs/7559355777/job/20582987874 and caused by merging of #11474
In intel#11474 it was noted in the documentation that the `prefetch` and `memadvise` handler methods were not supported as graph nodes on the OpenCL command-buffer backend. However, we never disabled the E2E tests for this target, which wasn't picked up by CI, as CI doesn't use the command-buffer extension in its OpenCL config. Correct this oversight.
In #11474 it was noted in the documentation that the `prefetch` and `memadvise` handler methods were not supported as graph nodes on the OpenCL command-buffer backend. However, we never disabled the E2E tests for this target, which wasn't picked up by CI, as CI doesn't use the command-buffer extension in its OpenCL config. This PR corrects this oversight by marking the tests as unsupported for OpenCL.
The merged PR intel#11472 added support for `handler::fill` and `handler::memset` nodes in a graph, while merged PR intel#11474 added support for `handler::prefetch` and `handler::mem_advise`.
Adds support for
prefetch
andadvise
memory hints.Adds e2e tests that verify that backend functions are called as they should be.