shafik added inline comments.
================ Comment at: clang/lib/AST/Interp/Integral.h:215 + static bool band(Integral A, Integral B, unsigned OpBits, Integral *R) { + *R = Integral(A.V & B.V); ---------------- Maybe `bitAnd`? ================ Comment at: clang/test/AST/Interp/literals.cpp:288 + + static_assert((1337 & -1) == 1337, ""); +}; ---------------- Also `0 &` something else should be `0`. I wonder if there is any value in using binary literals. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135012/new/ https://reviews.llvm.org/D135012 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits