[clang] [clang][Interp] IntegralAP zero-init (PR #68081)

Timm Baeder via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 14 00:13:25 PDT 2023


================
@@ -100,12 +100,13 @@ template <bool Signed> class IntegralAP final {
   }
   static IntegralAP from(const Boolean &B) {
     assert(false);
-    return IntegralAP::zero();
+    return IntegralAP::zero(1);
----------------
tbaederr wrote:

I don't have such comments because I opted for `assert(false)` where things are unimplemented :)

I keep opening smaller PRs _to make review easier_ because I thought that... makes it easier. If larger (but complete) patches are better for you, I can try to do that instead.

https://github.com/llvm/llvm-project/pull/68081


More information about the cfe-commits mailing list