Skip to content

[Driver][NFC] Update test to not pull in device libraries#22695

Merged
uditagarwal97 merged 1 commit into
intel:syclfrom
mdtoguchi:san-tst-updates-driver
Jul 21, 2026
Merged

[Driver][NFC] Update test to not pull in device libraries#22695
uditagarwal97 merged 1 commit into
intel:syclfrom
mdtoguchi:san-tst-updates-driver

Conversation

@mdtoguchi

Copy link
Copy Markdown
Contributor

This test does basic checks with -### and does an additional compilation that generates a preprocessed file to test for potential triple mismatch when extracting PCH files. When this is tested without any device libraries the compilation fails due to errors. Update the RUN lines to not pull in any device libraries. These are not important for the test.

This test does basic checks with -### and does an additional compilation
that generates a preprocessed file to test for potential triple mismatch
when extracting PCH files.  When this is tested without any device libraries
the compilation fails due to errors.  Update the RUN lines to not pull in
any device libraries.  These are not important for the test.
@mdtoguchi
mdtoguchi requested a review from a team as a code owner July 20, 2026 21:52
@mdtoguchi
mdtoguchi marked this pull request as draft July 20, 2026 22:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Updates the SYCL Windows effective triple driver test to avoid pulling in device libraries, preventing compilation failures when device libs aren’t available while keeping the test focused on effective triple/PCH extraction behavior.

Changes:

  • Adds --no-offloadlib and -fno-sycl-instrument-device-code to the relevant RUN: invocations.
  • Keeps the existing FileCheck assertions intact while adjusting driver invocation inputs.

@mdtoguchi
mdtoguchi marked this pull request as ready for review July 21, 2026 00:02
@sys-ce-bb

Copy link
Copy Markdown
Contributor

@intel/llvm-gatekeepers please consider merging

@uditagarwal97
uditagarwal97 merged commit 8443cbc into intel:sycl Jul 21, 2026
31 checks passed
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.

5 participants