Skip to content

[SYCL][Test] Make get_invalid_kernel_id.cpp linker-agnostic#22596

Merged
againull merged 3 commits into
intel:syclfrom
wenju-he:fix-get_invalid_kernel_id.cpp
Jul 14, 2026
Merged

[SYCL][Test] Make get_invalid_kernel_id.cpp linker-agnostic#22596
againull merged 3 commits into
intel:syclfrom
wenju-he:fix-get_invalid_kernel_id.cpp

Conversation

@wenju-he

@wenju-he wenju-he commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

CHECK-LINUX previously matched only ld.bfd's "undefined reference to" wording. Under parallel test execution, clang can select lld instead, which emits "undefined symbol:" with different punctuation, causing spurious FileCheck failures (test passes in isolation, flaky under check-sycl -j and on buildbot). Match both linker's phrasing.

Fix flaky fail in check-sycl:
ld: error: undefined symbol: sycl::_V1::detail::getKernelNameHelper(sycl::_V1::detail::KernelIdentity)
CMPLRLLVM-76620

CHECK-LINUX previously matched only ld.bfd's "undefined reference to"
wording. Under parallel test execution, clang can select lld instead,
which emits "undefined symbol:" with different punctuation, causing
spurious FileCheck failures (test passes in isolation, flaky under
check-sycl -j and on buildbot). Match both linker's phrasing.

Fix flaky fail in check-sycl.
CMPLRLLVM-76620

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
@wenju-he wenju-he requested a review from a team as a code owner July 10, 2026 09:21
@wenju-he wenju-he requested a review from cperkinsintel July 10, 2026 09:21
@wenju-he

Copy link
Copy Markdown
Contributor Author

@intel/llvm-gatekeepers please merge, thanks.

Timed Out Tests (1):
  SYCL :: Assert/assert_in_simultaneously_multiple_tus.cpp

is known issue: #21023

@againull againull merged commit 6fc5c4b into intel:sycl Jul 14, 2026
30 of 31 checks passed
@wenju-he wenju-he deleted the fix-get_invalid_kernel_id.cpp branch July 14, 2026 00:35
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.

3 participants