Skip to content

[SYCL] Fix abi/layout_* tests after enum diagnostic printing change#22697

Merged
jsji merged 1 commit into
intel:sycl-webfrom
wenju-he:fix-abi-layout_accessors_host.cpp
Jul 21, 2026
Merged

[SYCL] Fix abi/layout_* tests after enum diagnostic printing change#22697
jsji merged 1 commit into
intel:sycl-webfrom
wenju-he:fix-abi-layout_accessors_host.cpp

Conversation

@wenju-he

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

Copy link
Copy Markdown
Contributor

Commit bdfcef5 ("[clang] improve diagnostics for enums") made clang print enum-typed template arguments using their enumerator name rather than a C-style cast, changing -fdump-record-layouts output from _Sp_counted_base<(enum __gnu_cxx::_Lock_policy)2U> to _Sp_counted_base<__gnu_cxx::_S_atomic>.

CMPLRLLVM-77064

Commit bdfcef5 ("[clang] improve diagnostics for enums") made
clang print enum-typed template arguments using their enumerator name
rather than a C-style cast, changing -fdump-record-layouts output from
_Sp_counted_base<(enum __gnu_cxx::_Lock_policy)2U> to
_Sp_counted_base<__gnu_cxx::_S_atomic>. Update the CHECK lines to match.

CMPLRLLVM-77064

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@wenju-he
wenju-he requested a review from a team as a code owner July 21, 2026 04:31
@wenju-he
wenju-he requested review from sergey-semenov and removed request for a team July 21, 2026 04:31
@jsji
jsji merged commit 84cc6db into intel:sycl-web Jul 21, 2026
@wenju-he
wenju-he deleted the fix-abi-layout_accessors_host.cpp branch July 21, 2026 21:57
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