[llvm] r338236 - Revert "[X86] Correct the immediate cost for 'add/sub i64 %x, 0x80000000'."

Dean Michael Berris via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 30 02:45:10 PDT 2018


Author: dberris
Date: Mon Jul 30 02:45:09 2018
New Revision: 338236

URL: http://llvm.org/viewvc/llvm-project?rev=338236&view=rev
Log:
Revert "[X86] Correct the immediate cost for 'add/sub i64 %x, 0x80000000'."

This reverts commit r338204.

Modified:
    llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp

Modified: llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp?rev=338236&r1=338235&r2=338236&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp Mon Jul 30 02:45:09 2018
@@ -2332,15 +2332,9 @@ int X86TTIImpl::getIntImmCost(unsigned O
     // immediates here as the normal path expects bit 31 to be sign extended.
     if (Idx == 1 && Imm.getBitWidth() == 64 && isUInt<32>(Imm.getZExtValue()))
       return TTI::TCC_Free;
-    ImmIdx = 1;
-    break;
+    LLVM_FALLTHROUGH;
   case Instruction::Add:
   case Instruction::Sub:
-    // For add/sub, we can use the opposite instruction for INT32_MIN.
-    if (Idx == 1 && Imm.getBitWidth() == 64 && isInt<32>(-Imm.getSExtValue()))
-      return TTI::TCC_Free;
-    ImmIdx = 1;
-    break;
   case Instruction::Mul:
   case Instruction::UDiv:
   case Instruction::SDiv:




More information about the llvm-commits mailing list