[PATCH] D142609: [Clang] Fix -Wconstant-logical-operand when LHS is a constant
Nick Desaulniers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 16 10:44:15 PST 2023
nickdesaulniers added a comment.
Hmm...looking at some of the commits to the related code, it might be very intentional that we don't warn symmetrically:
938533db602b32ab435078e723b656ac6e779a1b
e54ff6cc3e479523b71e4c7eb4bd13707d84de0f
================
Comment at: clang/test/SemaCXX/expressions.cpp:146-148
#define Y2 2
bool r2 = X || Y2; // expected-warning {{use of logical '||' with constant operand}} \
// expected-note {{use '|' for a bitwise operation}}
----------------
So I think we'll want to change this test.
See commit d6eb2b9f4d4fc236376e3a5a7b8faa31e8dd427d that introduced it.
If we have a constant that was defined via macro, we DONT want to warn for it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142609/new/
https://reviews.llvm.org/D142609
More information about the cfe-commits
mailing list