[llvm] 0df3200 - [ValueTracking] Fix KnownBits conflict for poison-only vector
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 21 00:23:56 PST 2023
Author: Nikita Popov
Date: 2023-12-21T09:23:47+01:00
New Revision: 0df320093155f937ae985847f20e28b790667a9f
URL: https://github.com/llvm/llvm-project/commit/0df320093155f937ae985847f20e28b790667a9f
DIFF: https://github.com/llvm/llvm-project/commit/0df320093155f937ae985847f20e28b790667a9f.diff
LOG: [ValueTracking] Fix KnownBits conflict for poison-only vector
If all the demanded elements are poison, return unknown instead of
conflict to avoid downstream assertions.
Fixes https://github.com/llvm/llvm-project/issues/75505.
Added:
llvm/test/Analysis/ValueTracking/pr75505.ll
Modified:
llvm/lib/Analysis/ValueTracking.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Analysis/ValueTracking.cpp b/llvm/lib/Analysis/ValueTracking.cpp
index 9b8376833d2e65..efdb3fc2858249 100644
--- a/llvm/lib/Analysis/ValueTracking.cpp
+++ b/llvm/lib/Analysis/ValueTracking.cpp
@@ -1826,6 +1826,8 @@ void computeKnownBits(const Value *V, const APInt &DemandedElts,
Known.Zero &= ~Elt;
Known.One &= Elt;
}
+ if (Known.hasConflict())
+ Known.resetAll();
return;
}
@@ -1849,6 +1851,8 @@ void computeKnownBits(const Value *V, const APInt &DemandedElts,
Known.Zero &= ~Elt;
Known.One &= Elt;
}
+ if (Known.hasConflict())
+ Known.resetAll();
return;
}
diff --git a/llvm/test/Analysis/ValueTracking/pr75505.ll b/llvm/test/Analysis/ValueTracking/pr75505.ll
new file mode 100644
index 00000000000000..79368ff18e0604
--- /dev/null
+++ b/llvm/test/Analysis/ValueTracking/pr75505.ll
@@ -0,0 +1,17 @@
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py UTC_ARGS: --version 4
+; RUN: opt -S -passes=instsimplify < %s | FileCheck %s
+
+; Just make sure that we don't assert.
+define i32 @test(<2 x i16> %a, i32 %b) {
+; CHECK-LABEL: define i32 @test(
+; CHECK-SAME: <2 x i16> [[A:%.*]], i32 [[B:%.*]]) {
+; CHECK-NEXT: [[MUL:%.*]] = mul <2 x i16> [[A]], <i16 -1, i16 poison>
+; CHECK-NEXT: [[BC:%.*]] = bitcast <2 x i16> [[MUL]] to i32
+; CHECK-NEXT: [[LSHR:%.*]] = lshr i32 [[B]], [[BC]]
+; CHECK-NEXT: ret i32 [[LSHR]]
+;
+ %mul = mul <2 x i16> %a, <i16 -1, i16 poison>
+ %bc = bitcast <2 x i16> %mul to i32
+ %lshr = lshr i32 %b, %bc
+ ret i32 %lshr
+}
More information about the llvm-commits
mailing list