xbolva00 marked an inline comment as done.
xbolva00 added inline comments.

================
Comment at: lib/Sema/SemaExpr.cpp:10983
+
+    S.Diag(Loc, diag::note_xor_used_as_pow_silence) << ("0x2 ^ " + RHSStr);
+  } else if (LeftSideValue == 10) {
----------------
Quuxplusone wrote:
> Do you currently warn on, let's say, `2uLL ^ 7`? and if so, do we care that 
> the suggested expression `0x2 ^ 7` has a different type?  I imagine the 
> answer to "do we care" is "no," but I thought I should ask.
We check if bidwith is same.. Bail out if not.


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

Reply via email to