Skip to content

Commit f34d3e8

Browse files
goldsteinnronlieb
authored andcommitted
[ValueTracking] Consistently propagate DemandedElts is isKnownNonZero
Change-Id: Ie0c7814e68d7ad80022e128f3bceb906bb92fa0b
1 parent c669844 commit f34d3e8

File tree

2 files changed

+1
-7
lines changed

2 files changed

+1
-7
lines changed

llvm/lib/Analysis/ValueTracking.cpp

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2777,11 +2777,8 @@ static bool isKnownNonZeroFromOperator(const Operator *I,
27772777
// This all implies the 2 i16 elements are non-zero.
27782778
Type *FromTy = I->getOperand(0)->getType();
27792779
if ((FromTy->isIntOrIntVectorTy() || FromTy->isPtrOrPtrVectorTy()) &&
2780-
(BitWidth % getBitWidth(FromTy->getScalarType(), Q.DL)) == 0) {
2781-
if (match(I, m_ElementWiseBitCast(m_Value())))
2782-
return isKnownNonZero(I->getOperand(0), DemandedElts, Q, Depth);
2780+
(BitWidth % getBitWidth(FromTy->getScalarType(), Q.DL)) == 0)
27832781
return isKnownNonZero(I->getOperand(0), Q, Depth);
2784-
}
27852782
} break;
27862783
case Instruction::IntToPtr:
27872784
// Note that we have to take special care to avoid looking through

revert_patches.txt

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,6 @@ Revert: more Sema:
9595
9e1f1cfa59c4 [Clang][Sema] Handle class member access expressions with valid nested-name-specifiers that become invalid after lookup (#98167)
9696
RonL
9797
---
98-
breaks rocALUTION build
99-
72ff0499bba7 [ValueTracking] Consistently propagate `DemandedElts` is `isKnownNonZero`
100-
--
10198
Revert: breaks devIO for openmp
10299
dfeb3991fb48 Remove the `x86_mmx` IR type. (#98505)
103100
b7e4fba6e5dc Cleanup x86_mmx after removing IR type (#100646) (Reason: dependent on dfeb3991fb48)

0 commit comments

Comments
 (0)