-
Notifications
You must be signed in to change notification settings - Fork 730
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][LIBCLC] Allow custom tools location when building libclc #12034
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.
Overall LGTM! Just a handful small nits.
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!
27f3bf2
to
e4f3832
Compare
Failures seem to be a blip in CI, will trigger again. |
Are we aware of any CI issues now, I see CUDA E2E failing: https://github.com/intel/llvm/actions/runs/7089054304/job/19292940897#step:2:119 |
We are aware of this failure in the AWS CUDA container. It first appeared around last Friday. |
e4f3832
to
e6c8df3
Compare
e6c8df3
to
b26c4aa
Compare
Use `LIBCLC_CUSTOM_LLVM_TOOLS_BINARY_DIR` to specify the location of custom toolchain to be used for creation of libclc. This helps with debug build times: * debug build of sycl-toolchain with libclc built with debug tools: ```sh $ for f in $(ls lib/clc/*.bc); touch $f; time ninja sycl-toolchain [0/2] Re-checking globbed directories... [6/6] Generating ../../lib/clc/remangled-l64-signed_char.libspirv-amdgcn-amd-amdhsa.bc ninja sycl-toolchain 682.55s user 1.33s system 112% cpu 10:07.81 total ``` * debug build of sycl-toolchain with libclc built with release tools: ```sh $ for f in $(ls lib/clc/*.bc); touch $f; time ninja sycl-toolchain [0/2] Re-checking globbed directories... [6/6] Generating ../../lib/clc/remangled-l64-signed_char.libspirv-amdgcn-amd-amdhsa.bc ninja sycl-toolchain 158.51s user 1.15s system 189% cpu 1:24.31 total ``` Fixes: #6925
Use
LIBCLC_CUSTOM_LLVM_TOOLS_BINARY_DIR
to specify the location of custom toolchain to be used for creation of libclc. This helps with debug build times:Fixes: #6925