-
Notifications
You must be signed in to change notification settings - Fork 745
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][NATIVECPU] Point oneAPI Construction Kit tag to main branch #12940
Conversation
@@ -23,32 +23,67 @@ add_sycl_plugin(native_cpu | |||
UnifiedRuntimeCommon | |||
) | |||
|
|||
option(OCK_SOURCE_DIR "Root to a local checkout of the oneAPI Construction Kit" "") | |||
option(OCK_GIT_REPO "Git repository for the oneAPI Construction Kit FetchContent" "") |
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.
I thought option was for a boolean variable https://cmake.org/cmake/help/latest/command/option.html
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.
Thanks, I wasn't really aware of that, I changed the non-bool ones to use set()
.
@intel/llvm-gatekeepers this looks ready to get merged, thank you |
This PR changes the git tag for the oneAPI Construction Kit, pointing to a commit in the
main
branch. It also adds CMake variables to either use a local checkout of the OCK, or overwrite the git repo/tag used byFetchContent
.