[llvm] [KnownBits] Remove `hasConflict()` assertions (PR #94568)
    Jay Foad via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jun  7 08:05:08 PDT 2024
    
    
  
================
@@ -653,8 +656,10 @@ TEST(KnownBitsTest, GetMinMaxVal) {
       Min = APIntOps::umin(Min, N);
       Max = APIntOps::umax(Max, N);
     });
-    EXPECT_EQ(Min, Known.getMinValue());
-    EXPECT_EQ(Max, Known.getMaxValue());
+    if (!Known.hasConflict()) {
----------------
jayfoad wrote:
If there's a conflict you could immediately "continue" at the top of the loop to save some time. Same in the next three functions below.
https://github.com/llvm/llvm-project/pull/94568
    
    
More information about the llvm-commits
mailing list