diff options
| author | Dan Gohman <gohman@apple.com> | 2009-02-24 02:00:40 +0000 |
|---|---|---|
| committer | Dan Gohman <gohman@apple.com> | 2009-02-24 02:00:40 +0000 |
| commit | 4f356bb9b0a9dfc6e3d195d13ba45b4cfc1ed4c1 (patch) | |
| tree | c121dab1093a2c0a8ee18730e801999f84d4adc6 /llvm/lib/Analysis/ValueTracking.cpp | |
| parent | 5d1f458f0fc996dd15c8047d358963fe623878e8 (diff) | |
| download | bcm5719-llvm-4f356bb9b0a9dfc6e3d195d13ba45b4cfc1ed4c1.tar.gz bcm5719-llvm-4f356bb9b0a9dfc6e3d195d13ba45b4cfc1ed4c1.zip | |
Fix a ValueTracking rule: RHS means operand 1, not 0. Add a simple
ashr instcombine to help expose this code. And apply the fix to
SelectionDAG's copy of this code too.
llvm-svn: 65364
Diffstat (limited to 'llvm/lib/Analysis/ValueTracking.cpp')
| -rw-r--r-- | llvm/lib/Analysis/ValueTracking.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Analysis/ValueTracking.cpp b/llvm/lib/Analysis/ValueTracking.cpp index 7ac138b90ef..20fa69ea24f 100644 --- a/llvm/lib/Analysis/ValueTracking.cpp +++ b/llvm/lib/Analysis/ValueTracking.cpp @@ -625,7 +625,7 @@ unsigned llvm::ComputeNumSignBits(Value *V, TargetData *TD, unsigned Depth) { if (Tmp == 1) return 1; // Early out. // Special case decrementing a value (ADD X, -1): - if (ConstantInt *CRHS = dyn_cast<ConstantInt>(U->getOperand(0))) + if (ConstantInt *CRHS = dyn_cast<ConstantInt>(U->getOperand(1))) if (CRHS->isAllOnesValue()) { APInt KnownZero(TyBits, 0), KnownOne(TyBits, 0); APInt Mask = APInt::getAllOnesValue(TyBits); |
