Skip to content

[Clang][Test] Fix optnone-pragma-optimize-off.cpp after 4a8b9399eb9c - #22978

Closed
wenju-he wants to merge 1 commit into
intel:sycl-webfrom
wenju-he:fix-optnone-pragma-optimize-off.cpp
Closed

[Clang][Test] Fix optnone-pragma-optimize-off.cpp after 4a8b9399eb9c#22978
wenju-he wants to merge 1 commit into
intel:sycl-webfrom
wenju-he:fix-optnone-pragma-optimize-off.cpp

Conversation

@wenju-he

Copy link
Copy Markdown
Contributor

Upstream commit 4a8b939 ("[Clang][Sema] Fix lambda attribute processing order with clang optimize off") added a check for an always_inline lambda declared in the initializer of a namespace-scope variable under #pragma clang optimize off.

intel/llvm intentionally diverges from upstream in Sema::getCurrentMangleNumberContext(): commit 478c205 ("[SYCL] Fix Lambda Mangling in Namespace-Scope Variable Initializers", #20176) cherry-picks community PR llvm/llvm-project#159115, which mangles the the closure type with a closure-prefix for the enclosing variable.

CMPLRLLVM-77788

Upstream commit 4a8b939 ("[Clang][Sema] Fix lambda attribute
processing order with clang optimize off") added a check for an
always_inline lambda declared in the initializer of a namespace-scope
variable under `#pragma clang optimize off`.

intel/llvm intentionally diverges from upstream in
Sema::getCurrentMangleNumberContext(): commit 478c205 ("[SYCL] Fix
Lambda Mangling in Namespace-Scope Variable Initializers", intel#20176)
cherry-picks community PR llvm/llvm-project#159115, which mangles the
the closure type with a closure-prefix for the enclosing variable.

CMPLRLLVM-77788

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

Copy link
Copy Markdown
Contributor Author

same fix was accidently merged into sycl-web by merge commit 9c508e7 while fixing other check-all fails.
Close.

@wenju-he wenju-he closed this Aug 21, 2026
@wenju-he
wenju-he deleted the fix-optnone-pragma-optimize-off.cpp branch August 21, 2026 10:50
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.

2 participants