aaron.ballman added inline comments.

================
Comment at: test/Misc/ast-dump-stmt.c:66
+  // CHECK:        ImplicitCastExpr
+  // CHECK:          DeclRefExpr{{.*}}'T2' 'int'
+}
----------------
efriedma wrote:
> aaron.ballman wrote:
> > efriedma wrote:
> > > What does it mean for bitwise complement to "overflow"?
> > When the sign bit flips on a signed value, e.g., `~0`.
> Bitwise complement always flips the sign bit; calling that an "overflow" 
> doesn't seem useful.
For reference, I am upstreaming some changes we needed for an out-of-tree 
project, so "useful" kinda depends on context. However, I looked at our uses of 
this functionality and I'm not convinced we need this to report overflows, so 
I've made the change to not report overflow on a one's complement operand.


https://reviews.llvm.org/D33563



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to