[PATCH] D134744: [clang][Interp] Implement rem opcode

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 28 05:10:08 PDT 2022


tbaeder added inline comments.


================
Comment at: clang/test/AST/Interp/literals.cpp:161
+                                        // expected-note {{division by zero}}
+
+
----------------
aaron.ballman wrote:
> Same question here as with div in regards to testing float behavior. (If you don't intend to support floats yet, make sure the commit message makes that clear.)
Ultimately sure, but as of right now, floats aren't supported at all, so nope.


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

https://reviews.llvm.org/D134744



More information about the cfe-commits mailing list