diff options
author | Sanjay Patel <spatel@rotateright.com> | 2018-11-03 13:18:55 +0000 |
---|---|---|
committer | Sanjay Patel <spatel@rotateright.com> | 2018-11-03 13:18:55 +0000 |
commit | cac28b452e4703ff7789b41476b1349d3894ad0d (patch) | |
tree | 74eff88282359e57e9c0dd57e7fbef0bd6ab9d51 /llvm/lib/Analysis/ValueTracking.cpp | |
parent | ddcb0e4498e61dc6b5151f5c137e0857148a3d4f (diff) | |
download | llvm-cac28b452e4703ff7789b41476b1349d3894ad0d.zip llvm-cac28b452e4703ff7789b41476b1349d3894ad0d.tar.gz llvm-cac28b452e4703ff7789b41476b1349d3894ad0d.tar.bz2 |
[ValueTracking] peek through 2-input shuffles in ComputeNumSignBits
This patch gives the IR ComputeNumSignBits the same functionality as the
DAG version (the code is derived from the existing code).
This an extension of the single input shuffle analysis added with D53659.
Differential Revision: https://reviews.llvm.org/D53987
llvm-svn: 346071
Diffstat (limited to 'llvm/lib/Analysis/ValueTracking.cpp')
-rw-r--r-- | llvm/lib/Analysis/ValueTracking.cpp | 53 |
1 files changed, 34 insertions, 19 deletions
diff --git a/llvm/lib/Analysis/ValueTracking.cpp b/llvm/lib/Analysis/ValueTracking.cpp index 89a6215..6e08272 100644 --- a/llvm/lib/Analysis/ValueTracking.cpp +++ b/llvm/lib/Analysis/ValueTracking.cpp @@ -2512,26 +2512,41 @@ static unsigned ComputeNumSignBitsImpl(const Value *V, unsigned Depth, return ComputeNumSignBits(U->getOperand(0), Depth + 1, Q); case Instruction::ShuffleVector: { - // If the shuffle mask contains any undefined elements, that element of the - // result is undefined. Propagating information from a source operand may - // not be correct in that case, so just bail out. - if (cast<ShuffleVectorInst>(U)->getMask()->containsUndefElement()) - break; - - // If everything is undef, we can't say anything. This should be simplified. - Value *Op0 = U->getOperand(0), *Op1 = U->getOperand(1); - if (isa<UndefValue>(Op0) && isa<UndefValue>(Op1)) + // TODO: This is copied almost directly from the SelectionDAG version of + // ComputeNumSignBits. It would be better if we could share common + // code. If not, make sure that changes are translated to the DAG. + + // Collect the minimum number of sign bits that are shared by every vector + // element referenced by the shuffle. + auto *Shuf = cast<ShuffleVectorInst>(U); + int NumElts = Shuf->getOperand(0)->getType()->getVectorNumElements(); + int NumMaskElts = Shuf->getMask()->getType()->getVectorNumElements(); + APInt DemandedLHS(NumElts, 0), DemandedRHS(NumElts, 0); + for (int i = 0; i != NumMaskElts; ++i) { + int M = Shuf->getMaskValue(i); + assert(M < NumElts * 2 && "Invalid shuffle mask constant"); + // For undef elements, we don't know anything about the common state of + // the shuffle result. + if (M == -1) + return 1; + if (M < NumElts) + DemandedLHS.setBit(M % NumElts); + else + DemandedRHS.setBit(M % NumElts); + } + Tmp = std::numeric_limits<unsigned>::max(); + if (!!DemandedLHS) + Tmp = ComputeNumSignBits(Shuf->getOperand(0), Depth + 1, Q); + if (!!DemandedRHS) { + Tmp2 = ComputeNumSignBits(Shuf->getOperand(1), Depth + 1, Q); + Tmp = std::min(Tmp, Tmp2); + } + // If we don't know anything, early out and try computeKnownBits fall-back. + if (Tmp == 1) break; - - // Look through shuffle of 1 source vector. - if (isa<UndefValue>(Op0)) - return ComputeNumSignBits(Op1, Depth + 1, Q); - if (isa<UndefValue>(Op1)) - return ComputeNumSignBits(Op0, Depth + 1, Q); - - // TODO: We can look through shuffles of 2 sources by computing the minimum - // sign bits for each operand (similar to what we do for binops). - break; + assert(Tmp <= V->getType()->getScalarSizeInBits() && + "Failed to determine minimum sign bits"); + return Tmp; } } |