diff options
| author | Chris Lattner <sabre@nondot.org> | 2009-01-09 06:08:12 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2009-01-09 06:08:12 +0000 |
| commit | 6b644ff7ba97525c9f9e2367dacb97ecd5ead385 (patch) | |
| tree | 7f3ce7e038b3926f08c48bcce721c39af9b956c0 /lib/Transforms | |
| parent | c76d64383c08f60cd7c96e751ae97123a2b09e67 (diff) | |
| download | external_llvm-6b644ff7ba97525c9f9e2367dacb97ecd5ead385.zip external_llvm-6b644ff7ba97525c9f9e2367dacb97ecd5ead385.tar.gz external_llvm-6b644ff7ba97525c9f9e2367dacb97ecd5ead385.tar.bz2 | |
Fix PR3298, a crash in Jump Threading. Apparently even
jump threading can have bugs, who knew? ;-)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61983 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Scalar/JumpThreading.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/Transforms/Scalar/JumpThreading.cpp b/lib/Transforms/Scalar/JumpThreading.cpp index 59af14b..94d33aa 100644 --- a/lib/Transforms/Scalar/JumpThreading.cpp +++ b/lib/Transforms/Scalar/JumpThreading.cpp @@ -504,6 +504,10 @@ bool JumpThreading::SimplifyPartiallyRedundantLoad(LoadInst *LI) { // If the value if the load is locally available within the block, just use // it. This frequently occurs for reg2mem'd allocas. //cerr << "LOAD ELIMINATED:\n" << *BBIt << *LI << "\n"; + + // If the returned value is the load itself, replace with an undef. This can + // only happen in dead loops. + if (AvailableVal == LI) AvailableVal = UndefValue::get(LI->getType()); LI->replaceAllUsesWith(AvailableVal); LI->eraseFromParent(); return true; |