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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
