From fd2934f190fc13b60f8e5dd9639008d721b170f8 Mon Sep 17 00:00:00 2001 From: Eli Friedman Date: Wed, 15 Jul 2009 22:13:34 +0000 Subject: [PATCH] Don't restrict the set of instructions where we try to constant-fold the operands; it's possible to end up with a constant-foldable operand to most instructions, even those which can't trap. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@75845 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/InstructionCombining.cpp | 4 +--- test/Transforms/InstCombine/fold-bin-operand.ll | 13 +++++++++++++ 2 files changed, 14 insertions(+), 3 deletions(-) create mode 100644 test/Transforms/InstCombine/fold-bin-operand.ll diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 4160ee7dc1e..fd87e30f963 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -13021,9 +13021,7 @@ bool InstCombiner::DoOneIteration(Function &F, unsigned Iteration) { continue; } - if (TD && - (I->getType()->getTypeID() == Type::VoidTyID || - I->isTrapping())) { + if (TD) { // See if we can constant fold its operands. for (User::op_iterator i = I->op_begin(), e = I->op_end(); i != e; ++i) if (ConstantExpr *CE = dyn_cast(i)) diff --git a/test/Transforms/InstCombine/fold-bin-operand.ll b/test/Transforms/InstCombine/fold-bin-operand.ll new file mode 100644 index 00000000000..e884e1948e7 --- /dev/null +++ b/test/Transforms/InstCombine/fold-bin-operand.ll @@ -0,0 +1,13 @@ +; RUN: llvm-as < %s | opt -instcombine | llvm-dis | not grep icmp + +define i1 @f(i1 %x) { + %b = and i1 %x, icmp eq (i8* inttoptr (i32 1 to i8*), i8* inttoptr (i32 2 to i8*)) + ret i1 %b +} + +; FIXME: This doesn't fold at the moment! +; define i32 @f(i32 %x) { +; %b = add i32 %x, zext (i1 icmp eq (i8* inttoptr (i32 1000000 to i8*), i8* inttoptr (i32 2000000 to i8*)) to i32) +; ret i32 %b +;} + -- 2.34.1