diff options
| author | Chris Lattner <sabre@nondot.org> | 2004-05-10 15:15:55 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2004-05-10 15:15:55 +0000 |
| commit | c81e6bae88d0b23c5d23b0b0ebcfa05c8de71af9 (patch) | |
| tree | ba47abd9080164b574d23459d9370284d2f0a6ae /lib/Target/X86/InstSelectSimple.cpp | |
| parent | 3699e05fb21d76dcce43462c7d9684342bf7b0ce (diff) | |
Fix a fairly serious pessimizaion that was preventing us from efficiently
compiling things like 'add long %X, 1'. The problem is that we were switching
the order of the operands for longs even though we can't fold them yet.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@13451 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/X86/InstSelectSimple.cpp')
| -rw-r--r-- | lib/Target/X86/InstSelectSimple.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/Target/X86/InstSelectSimple.cpp b/lib/Target/X86/InstSelectSimple.cpp index 38503886de..8fbeeb7f2e 100644 --- a/lib/Target/X86/InstSelectSimple.cpp +++ b/lib/Target/X86/InstSelectSimple.cpp @@ -1863,13 +1863,14 @@ void ISel::visitSimpleBinary(BinaryOperator &B, unsigned OperatorClass) { unsigned DestReg = getReg(B); MachineBasicBlock::iterator MI = BB->end(); Value *Op0 = B.getOperand(0), *Op1 = B.getOperand(1); + unsigned Class = getClassB(B.getType()); // Special case: op Reg, load [mem] - if (isa<LoadInst>(Op0) && !isa<LoadInst>(Op1)) + if (isa<LoadInst>(Op0) && !isa<LoadInst>(Op1) && Class != cLong && + isSafeToFoldLoadIntoInstruction(*cast<LoadInst>(Op0), B)) if (!B.swapOperands()) std::swap(Op0, Op1); // Make sure any loads are in the RHS. - unsigned Class = getClassB(B.getType()); if (isa<LoadInst>(Op1) && Class != cLong && isSafeToFoldLoadIntoInstruction(*cast<LoadInst>(Op1), B)) { |