Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Backport unreachable fix for C++, regressed in LLVM 18 #87694

Closed
ian-twilightcoder opened this issue Apr 4, 2024 · 2 comments
Closed

Backport unreachable fix for C++, regressed in LLVM 18 #87694

ian-twilightcoder opened this issue Apr 4, 2024 · 2 comments
Labels
clang:headers Headers provided by Clang, e.g. for intrinsics release:backport
Milestone
LLVM 18.X Release

Comments

@ian-twilightcoder
Copy link
Contributor

We should backport #86748 because it fixes a conflicting unreachable macro in C++ in some situations, which were introduced in clang in LLVM 18.

/cherry-pick df69a30
@ian-twilightcoder ian-twilightcoder added this to the LLVM 18.X Release milestone Apr 4, 2024
@github-project-automation github-project-automation bot moved this to Needs Triage in LLVM Release Status Apr 4, 2024
@ian-twilightcoder
Copy link
Contributor Author

This is another C++ regression that was caused by https://reviews.llvm.org/D157757

@llvmbot llvmbot closed this as completed Apr 4, 2024
@github-project-automation github-project-automation bot moved this from Needs Triage to Done in LLVM Release Status Apr 4, 2024
@llvmbot
Copy link
Member

llvmbot commented Apr 4, 2024

/pull-request #87696

@EugeneZelenko EugeneZelenko added clang:headers Headers provided by Clang, e.g. for intrinsics release:backport and removed new issue labels Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clang:headers Headers provided by Clang, e.g. for intrinsics release:backport
Projects
LLVM Release Status
Status: Done
Milestone
LLVM 18.X Release
Development

No branches or pull requests
3 participants
@EugeneZelenko @ian-twilightcoder @llvmbot