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

PR for llvm/llvm-project#64515 #546

Merged
merged 1 commit into from
Aug 9, 2023
Merged

PR for llvm/llvm-project#64515 #546

merged 1 commit into from
Aug 9, 2023

Conversation

llvmbot
Copy link
Collaborator

@llvmbot llvmbot commented Aug 8, 2023

This reverts commit dfdfd30.

An issue is reported for wrong warning, this has to be reconsidered.

Differential Revision: https://reviews.llvm.org/D157352
@tru
Copy link
Contributor

tru commented Aug 8, 2023

@nickdesaulniers should this be merged to the release branch?

Copy link
Contributor

@AaronBallman AaronBallman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@nickdesaulniers
Copy link
Member

LGTM

@tru tru merged commit a93ca35 into release/17.x Aug 9, 2023
8 of 9 checks passed
@tru tru deleted the xgupta-release/17.x branch August 9, 2023 06:58
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.

backport a84525233776a716e2c6291993f0b33fd1c76f7c (Wrong -Wconstant-logical-operand warning)
5 participants