aboutsummaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorChris Lattner <sabre@nondot.org>2008-06-20 05:12:56 +0000
committerChris Lattner <sabre@nondot.org>2008-06-20 05:12:56 +0000
commit476983ab8cc765cfb88c9b49d82edf5c4ab85e25 (patch)
tree3689de4d4c8aa5c55b18b2bbbc8c1a857a28f808 /lib
parent1d5610ad4d7469c6bc997c509a590d94f36258e8 (diff)
downloadexternal_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.cpp8
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;