-
Notifications
You must be signed in to change notification settings - Fork 769
[SYCL] Rename SYCL_ENABLE_PLUGINS to SYCL_ENABLE_BACKENDS #14931
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] Rename SYCL_ENABLE_PLUGINS to SYCL_ENABLE_BACKENDS #14931
Conversation
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.
doc/
changes LGTM
Also rename SYCL_PI_UR_USE_FETCH_CONTENT to SYCL_UR_USE_FETCH_CONTENT and SYCL_PI_UR_SOURCE_DIR to SYCL_UR_SOURCE_DIR.
Tell user they should use SYCL_ENABLE_BACKENDS instead. Use the value given for the old flag for the new flag.
6c0d1d3
to
60ecb9d
Compare
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, thank you
@intel/dpcpp-devops-reviewers could someone from this team take a look at this PR please? thanks! |
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! Thanks for adding the safety-net-with-warning path!
@intel/llvm-gatekeepers this is now ready for merge, please and thanks! |
Rename
SYCL_ENABLE_PLUGINS
toSYCL_ENABLE_BACKENDS
Also rename
SYCL_PI_UR_USE_FETCH_CONTENT
toSYCL_UR_USE_FETCH_CONTENT
and
SYCL_PI_UR_SOURCE_DIR
toSYCL_UR_SOURCE_DIR
For #14927