From 2de3fec61c9e6b6376b3480936133d49e94c67a1 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Fri, 20 Jun 2008 05:12:56 +0000 Subject: [PATCH] Fix PR2471, which is a bug involving an invalid promotion from a conditional load. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@52525 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/InstructionCombining.cpp | 8 +++++++- .../InstCombine/2008-06-19-UncondLoad.ll | 16 ++++++++++++++++ 2 files changed, 23 insertions(+), 1 deletion(-) create mode 100644 test/Transforms/InstCombine/2008-06-19-UncondLoad.ll diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 72a27f500c9..54e611d2a1e 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -10062,10 +10062,16 @@ static bool isSafeToLoadUnconditionally(Value *V, Instruction *ScanFrom) { while (BBI != E) { --BBI; + // If we see a free or a call (which might do a free) the pointer could be + // marked invalid. + if (isa(BBI) || isa(BBI)) + return false; + if (LoadInst *LI = dyn_cast(BBI)) { if (LI->getOperand(0) == V) return true; - } else if (StoreInst *SI = dyn_cast(BBI)) + } else if (StoreInst *SI = dyn_cast(BBI)) { if (SI->getOperand(1) == V) return true; + } } return false; diff --git a/test/Transforms/InstCombine/2008-06-19-UncondLoad.ll b/test/Transforms/InstCombine/2008-06-19-UncondLoad.ll new file mode 100644 index 00000000000..fa3dc0ae141 --- /dev/null +++ b/test/Transforms/InstCombine/2008-06-19-UncondLoad.ll @@ -0,0 +1,16 @@ +; RUN: llvm-as < %s | opt -instcombine | llvm-dis | grep load | count 3 +; PR2471 + +declare i32 @x(i32*) +define i32 @b(i32* %a, i32* %b) { +entry: + %tmp1 = load i32* %a + %tmp3 = load i32* %b + %add = add i32 %tmp1, %tmp3 + %call = call i32 @x( i32* %a ) + %tobool = icmp ne i32 %add, 0 + ; not safe to turn into an uncond load + %cond = select i1 %tobool, i32* %b, i32* %a + %tmp8 = load i32* %cond + ret i32 %tmp8 +}