diff options
| author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2019-06-20 10:56:37 +0000 |
|---|---|---|
| committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2019-06-20 10:56:37 +0000 |
| commit | a4876282704d2eb88405778f402f34973162f835 (patch) | |
| tree | 90dfbb932e7171d0d8af64d2ddc6cfab0eda8739 /llvm/lib/CodeGen/SelectionDAG | |
| parent | e10a881c2b04cf7735fcd105a954c52fa900d64e (diff) | |
| download | bcm5719-llvm-a4876282704d2eb88405778f402f34973162f835.tar.gz bcm5719-llvm-a4876282704d2eb88405778f402f34973162f835.zip | |
[DAGCombine] Reduce scope of ShAmtVal variable. NFCI.
Fixes cppcheck warning.
Use the more capable getAPIntVal() instead of getZExtValue() as well since I'm here.
llvm-svn: 363921
Diffstat (limited to 'llvm/lib/CodeGen/SelectionDAG')
| -rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp index 33e339de354..5eee749d330 100644 --- a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp @@ -9840,14 +9840,13 @@ SDValue DAGCombiner::visitZERO_EXTEND(SDNode *N) { N0.getOperand(0).getOpcode() == ISD::ZERO_EXTEND && N0.hasOneUse()) { SDValue ShAmt = N0.getOperand(1); - unsigned ShAmtVal = cast<ConstantSDNode>(ShAmt)->getZExtValue(); if (N0.getOpcode() == ISD::SHL) { SDValue InnerZExt = N0.getOperand(0); // If the original shl may be shifting out bits, do not perform this // transformation. unsigned KnownZeroBits = InnerZExt.getValueSizeInBits() - InnerZExt.getOperand(0).getValueSizeInBits(); - if (ShAmtVal > KnownZeroBits) + if (cast<ConstantSDNode>(ShAmt)->getAPIntValue().ugt(KnownZeroBits)) return SDValue(); } |
