[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