[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