[llvm] [KnownBits] Remove `hasConflict()` assertions (PR #94568)
Jay Foad via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 7 08:00:47 PDT 2024
================
@@ -38,8 +38,10 @@ static void TestBinOpExhaustive(Fn1 PropagateFn, Fn2 EvalFn) {
APInt AB1R = PropagateFn(0, AOut, Known1Redacted, Known2Redacted);
APInt AB2R = PropagateFn(1, AOut, Known1Redacted, Known2Redacted);
- EXPECT_EQ(AB1, AB1R);
- EXPECT_EQ(AB2, AB2R);
+ if (!Known1Redacted.hasConflict() && !Known2Redacted.hasConflict()) {
----------------
jayfoad wrote:
Is this change still needed in this version of the patch?
https://github.com/llvm/llvm-project/pull/94568
More information about the llvm-commits
mailing list