xbolva00 marked an inline comment as done.
xbolva00 added inline comments.
================
Comment at: test/Sema/warn-xor-as-pow.c:45
+ // expected-note@-1 {{replace expression with '(2) ^ 32' to silence this
warning}}
+ res = 2 ^ 64; // expected-warning {{result of '2 ^ 64' is 66; maybe you mean
'1<<64', but shift count >= width of type}}
+ // expected-note@-1 {{replace expression with '(2) ^ 64' to silence this
warning}}
----------------
jfb wrote:
> This one hits a ceiling, we can't really suggest anything for this value IMO.
> Maybe we need to see if the user is doing `2^64 - 1`? In that case we can
> offer a suggestion.
Since this requires changes in other parts of code to catch 2^64 - 1, maybe we
can leave it as a follow-up patch?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63423/new/
https://reviews.llvm.org/D63423
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
