diff options
| author | Edwin Török <edwintorok@gmail.com> | 2009-05-23 17:29:48 +0000 |
|---|---|---|
| committer | Edwin Török <edwintorok@gmail.com> | 2009-05-23 17:29:48 +0000 |
| commit | a9bb5d91c0090d8492df3314237c97cbbccfc02e (patch) | |
| tree | 8bffda5d5211572322ad8c2a363f01f52bac765b /lib/CodeGen/SelectionDAG/DAGCombiner.cpp | |
| parent | 98178f48c7142f7ec79260ead343f0afb98cf686 (diff) | |
| download | external_llvm-a9bb5d91c0090d8492df3314237c97cbbccfc02e.zip external_llvm-a9bb5d91c0090d8492df3314237c97cbbccfc02e.tar.gz external_llvm-a9bb5d91c0090d8492df3314237c97cbbccfc02e.tar.bz2 | |
Fix PR4254.
The DAGCombiner created a negative shiftamount, stored in an
unsigned variable. Later the optimizer eliminated the shift entirely as being
undefined.
Example: (srl (shl X, 56) 48). ShiftAmt is 4294967288.
Fix it by checking that the shiftamount is positive, and storing in a signed
variable.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@72331 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/SelectionDAG/DAGCombiner.cpp')
| -rw-r--r-- | lib/CodeGen/SelectionDAG/DAGCombiner.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp index 6e0cc9f..a866cb5 100644 --- a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp +++ b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp @@ -2546,13 +2546,13 @@ SDValue DAGCombiner::visitSRA(SDNode *N) { MVT TruncVT = MVT::getIntegerVT(VTValSize - N1C->getZExtValue()); // Determine the residual right-shift amount. - unsigned ShiftAmt = N1C->getZExtValue() - N01C->getZExtValue(); + signed ShiftAmt = N1C->getZExtValue() - N01C->getZExtValue(); // If the shift is not a no-op (in which case this should be just a sign // extend already), the truncated to type is legal, sign_extend is legal // on that type, and the the truncate to that type is both legal and free, // perform the transform. - if (ShiftAmt && + if ((ShiftAmt > 0) && TLI.isOperationLegalOrCustom(ISD::SIGN_EXTEND, TruncVT) && TLI.isOperationLegalOrCustom(ISD::TRUNCATE, VT) && TLI.isTruncateFree(VT, TruncVT)) { |