diff options
author | Chris Lattner <sabre@nondot.org> | 2008-06-20 05:12:56 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2008-06-20 05:12:56 +0000 |
commit | 476983ab8cc765cfb88c9b49d82edf5c4ab85e25 (patch) | |
tree | 3689de4d4c8aa5c55b18b2bbbc8c1a857a28f808 /lib | |
parent | 1d5610ad4d7469c6bc997c509a590d94f36258e8 (diff) | |
download | external_llvm-476983ab8cc765cfb88c9b49d82edf5c4ab85e25.zip external_llvm-476983ab8cc765cfb88c9b49d82edf5c4ab85e25.tar.gz external_llvm-476983ab8cc765cfb88c9b49d82edf5c4ab85e25.tar.bz2 |
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
Diffstat (limited to 'lib')
-rw-r--r-- | lib/Transforms/Scalar/InstructionCombining.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 72a27f5..54e611d 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<FreeInst>(BBI) || isa<CallInst>(BBI)) + return false; + if (LoadInst *LI = dyn_cast<LoadInst>(BBI)) { if (LI->getOperand(0) == V) return true; - } else if (StoreInst *SI = dyn_cast<StoreInst>(BBI)) + } else if (StoreInst *SI = dyn_cast<StoreInst>(BBI)) { if (SI->getOperand(1) == V) return true; + } } return false; |