[PATCH] D137071: [clang][Interp] Implement missing compound assign operators

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 8 11:04:19 PST 2022


aaron.ballman added a comment.

Precommit CI looks to have potentially found something interesting here.



================
Comment at: clang/test/AST/Interp/literals.cpp:553
+  static_assert(IntRem(2, 1) == 0, "");
+  static_assert(IntRem(9, 7) == 2, "");
+
----------------
aaron.ballman wrote:
> ```
> static_assert(IntRem(9, 0) == 12, ""); // Not constexpr
> static_assert(IntRem(__INT_MIN__, -1) == 12, ""); // Not constexpr
> ```
Missed the test for __INT_MIN__ and -1?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137071/new/

https://reviews.llvm.org/D137071



More information about the cfe-commits mailing list