From 6c5ac6de5c0605332cdb44218401b854ea8658c3 Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Mon, 22 Feb 2010 22:43:23 +0000 Subject: Canonicalize ConstantInts to the right operand of commutative operators. The test difference is just due to the multiplication operands being commuted (and thus requiring a more elaborate match). In optimized code, that expression would be folded. llvm-svn: 96816 --- llvm/lib/Analysis/ConstantFolding.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'llvm/lib/Analysis/ConstantFolding.cpp') diff --git a/llvm/lib/Analysis/ConstantFolding.cpp b/llvm/lib/Analysis/ConstantFolding.cpp index f3f9a5102f8..1d23fe0b6c9 100644 --- a/llvm/lib/Analysis/ConstantFolding.cpp +++ b/llvm/lib/Analysis/ConstantFolding.cpp @@ -794,8 +794,8 @@ Constant *llvm::ConstantFoldInstOperands(unsigned Opcode, const Type *DestTy, // it is casted back to a pointer, see if the expression can be // converted into a GEP. if (CE->getOpcode() == Instruction::Add) - if (ConstantInt *L = dyn_cast(CE->getOperand(0))) - if (ConstantExpr *R = dyn_cast(CE->getOperand(1))) + if (ConstantInt *L = dyn_cast(CE->getOperand(1))) + if (ConstantExpr *R = dyn_cast(CE->getOperand(0))) if (R->getOpcode() == Instruction::PtrToInt) if (GlobalVariable *GV = dyn_cast(R->getOperand(0))) { -- cgit v1.2.3