Skip to content

Conversation

@YixingZhang007
Copy link
Contributor

@YixingZhang007 YixingZhang007 commented Nov 20, 2025

This pull request resolves a typo introduced in a recent modification of clang/test/Driver/clang-linker-wrapper.cpp. In addition, the test logic is improved by replacing the use of CHK-NO-CMDS-AOT-GEN-NOT with CHK-NO-CMDS-AOT-GEN to verify the behavior where the device name should not be passed to sycl-post-link when clang-linker-wrapper is called with the --gpu-tool-arg=-device arch argument.

The previous PR introduced the mistake is #20470

@YixingZhang007 YixingZhang007 requested a review from a team as a code owner November 20, 2025 05:23
@YixingZhang007 YixingZhang007 changed the title Fix the typo in clang/test/Driver/clang-linker-wrapper.cpp [NewOffloadModel] Update clang-linker-wrapper test with issue fixes Nov 20, 2025
@YixingZhang007
Copy link
Contributor Author

I do not think the current failure in CI is related to the changes made in this PR :)

@YixingZhang007
Copy link
Contributor Author

@intel/llvm-gatekeepers The PR has been approved, and I don't think the current CI failure is caused by changes made in this PR. Could you please help merge this PR? Thank you :)

@sarnex sarnex merged commit 8ab28e0 into intel:sycl Nov 20, 2025
27 of 28 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.

4 participants