diff options
| author | Chris Lattner <sabre@nondot.org> | 2008-01-13 20:59:02 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2008-01-13 20:59:02 +0000 |
| commit | ee2b7a4530736d311fb3e2eae551e0acdb919354 (patch) | |
| tree | 58a288ca2539ed7d4cc29c698174f25b992b3a76 /lib/Transforms | |
| parent | 1fac17f2e7dafbbe2619bbd253dd7b957052f5e0 (diff) | |
| download | external_llvm-ee2b7a4530736d311fb3e2eae551e0acdb919354.zip external_llvm-ee2b7a4530736d311fb3e2eae551e0acdb919354.tar.gz external_llvm-ee2b7a4530736d311fb3e2eae551e0acdb919354.tar.bz2 | |
Fix PR1907, a nasty miscompilation because instcombine didn't
realize that ne & sgt was a signed comparison (it was only
looking at whether the left compare was signed).
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@45937 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Scalar/InstructionCombining.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index ea53aec..9465a31 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -3495,8 +3495,14 @@ Instruction *InstCombiner::visitAnd(BinaryOperator &I) { ICmpInst::isSignedPredicate(LHSCC) == ICmpInst::isSignedPredicate(RHSCC))) { // Ensure that the larger constant is on the RHS. - ICmpInst::Predicate GT = ICmpInst::isSignedPredicate(LHSCC) ? - ICmpInst::ICMP_SGT : ICmpInst::ICMP_UGT; + ICmpInst::Predicate GT; + if (ICmpInst::isSignedPredicate(LHSCC) || + (ICmpInst::isEquality(LHSCC) && + ICmpInst::isSignedPredicate(RHSCC))) + GT = ICmpInst::ICMP_SGT; + else + GT = ICmpInst::ICMP_UGT; + Constant *Cmp = ConstantExpr::getICmp(GT, LHSCst, RHSCst); ICmpInst *LHS = cast<ICmpInst>(Op0); if (cast<ConstantInt>(Cmp)->getZExtValue()) { |