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.

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

Backport b904e68f13ba7d4f4aa86a3495e2441c99247671 #61588

Closed
AaronBallman opened this issue Mar 21, 2023 · 7 comments · Fixed by llvm/llvm-project-release-prs#387
Closed

Backport b904e68f13ba7d4f4aa86a3495e2441c99247671 #61588

AaronBallman opened this issue Mar 21, 2023 · 7 comments · Fixed by llvm/llvm-project-release-prs#387
Labels
clang:diagnostics New/improved warning or error message in Clang, but not in clang-tidy or static analyzer release:backport release:merged

Comments

@AaronBallman
Copy link
Collaborator

/cherry-pick b904e68

@AaronBallman AaronBallman added release:backport clang:diagnostics New/improved warning or error message in Clang, but not in clang-tidy or static analyzer labels Mar 21, 2023
@AaronBallman AaronBallman added this to the LLVM 16.0.1 Release milestone Mar 21, 2023
@AaronBallman
Copy link
Collaborator Author

This addresses the second issue in #61582, which was identified after I had already fixed the first issue. So it relates to the backport at #61583 as well.

@llvmbot
Copy link
Collaborator

llvmbot commented Mar 21, 2023

Failed to cherry-pick: b904e68

https://github.com/llvm/llvm-project/actions/runs/4481712618

Please manually backport the fix and push it to your github fork. Once this is done, please add a comment like this:

/branch <user>/<repo>/<branch>

@AaronBallman
Copy link
Collaborator Author

Because these changes touch the release notes modified in #61583, I'm not certain how to proceed (I don't know if you want a situation where this PR relies on the other one being applied first, or not).

@AaronBallman
Copy link
Collaborator Author

Now that the other one has landed, I think this might cherry-pick cleanly now, so will try that again before doing the branch dance.

@AaronBallman
Copy link
Collaborator Author

/cherry-pick b904e68

@llvmbot
Copy link
Collaborator

llvmbot commented Mar 23, 2023

/branch llvm/llvm-project-release-prs/issue61588

@llvmbot
Copy link
Collaborator

llvmbot commented Mar 23, 2023

/pull-request llvm/llvm-project-release-prs#387

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:diagnostics New/improved warning or error message in Clang, but not in clang-tidy or static analyzer release:backport release:merged
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants