[llvm] [KnownBits] Remove `hasConflict()` assertions (PR #94568)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 7 09:02:01 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()) {
----------------
nikic wrote:
These particular APIs don't return KnownBits, so they don't really have any specific behavior they should exhibit for conflict inputs beyond "don't assert".
https://github.com/llvm/llvm-project/pull/94568
More information about the llvm-commits
mailing list