Skip to content

Conversation

wenju-he
Copy link
Contributor

@wenju-he wenju-he commented Sep 10, 2025

Since 388ccf5, tests fail when the libclc project is disabled
because the libspirv bitcode file is missing.
Fix them by adding -fno-sycl-libspirv flag.
Add -ccc-install-dir for libspirv specific tests.

The tests fail since 388ccf5 when libclc project is disabled, since
there is no libspirv bitcode file.
Fix them by adding `-fno-sycl-libspirv` flag.
Add `REQUIRES: libclc` to libspirv specific tests.
@wenju-he wenju-he requested review from a team as code owners September 10, 2025 02:16
@wenju-he wenju-he requested a review from Maetveis September 10, 2025 02:17
@wenju-he wenju-he changed the title [Driver][SYCL][NFC] Fix tests fails when libclc project is disabled [Driver][SYCL][Test] Fix tests fails when libclc project is disabled Sep 10, 2025
Copy link
Contributor

@Maetveis Maetveis left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the tests, see comments inline.

Copy link
Contributor

@Fznamznon Fznamznon left a comment

Choose a reason for hiding this comment

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

There are no real CFE changes, so hitting approval to unblock the merge

Copy link
Contributor

@hchilama hchilama left a comment

Choose a reason for hiding this comment

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

Changes looks good to me - Driver

@wenju-he
Copy link
Contributor Author

@intel/llvm-gatekeepers please merge, thanks. It is not clear to me why Jenkins/Precommit fails.

@sarnex sarnex merged commit a5c76b8 into intel:sycl Sep 11, 2025
27 of 28 checks passed
@wenju-he wenju-he deleted the fix-driver-libspirv-related-tests branch September 11, 2025 20:43
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.

6 participants