6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-01-28 21:34:23 +00:00
Code Issues Projects Releases Wiki Activity

LoopSimplify does not update domfrontier correctly.

Browse Source 
This fixes PR7649.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@108513 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Tobias Grosser 2010-07-16 05:59:45 +00:00
parent 9a7b37b8a3
commit 74c740052a
2 changed files with 22 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/VMCore/Dominators.cpp
View File

@ -148,11 +148,11 @@ void DominanceFrontier::splitBlock(BasicBlock *NewBB) {
} }
// If NewBB dominates NewBBSucc, then DF(NewBB) is now going to be the // If NewBB dominates NewBBSucc, then DF(NewBB) is now going to be the
// DF(PredBlocks[0]) without the stuff that the new block does not dominate // DF(NewBBSucc) without the stuff that the new block does not dominate
// a predecessor of. // a predecessor of.
DominatorTree &DT = getAnalysis<DominatorTree>(); DominatorTree &DT = getAnalysis<DominatorTree>();
if (DT.dominates(NewBB, NewBBSucc)) { if (DT.dominates(NewBB, NewBBSucc)) {
DominanceFrontier::iterator DFI = find(PredBlocks[0]); DominanceFrontier::iterator DFI = find(NewBBSucc);
if (DFI != end()) { if (DFI != end()) {
DominanceFrontier::DomSetType Set = DFI->second; DominanceFrontier::DomSetType Set = DFI->second;
// Filter out stuff in Set that we do not dominate a predecessor of. // Filter out stuff in Set that we do not dominate a predecessor of.

20
test/Transforms/LoopSimplify/2010-07-15-IncorrectDomFrontierUpdate.ll Normal file
View File

@ -0,0 +1,20 @@
; RUN: opt < %s -domfrontier -loopsimplify -domfrontier -verify-dom-info -analyze
define void @a() nounwind {
entry:
br i1 undef, label %bb37, label %bb1.i
bb1.i: ; preds = %bb1.i, %bb
%indvar = phi i64 [ %indvar.next, %bb1.i ], [ 0, %entry ] ; <i64> [#uses=1]
%indvar.next = add i64 %indvar, 1 ; <i64> [#uses=2]
%exitcond = icmp eq i64 %indvar.next, 576 ; <i1> [#uses=1]
br i1 %exitcond, label %bb37, label %bb1.i
bb37: ; preds = %bb1.i, %bb
br label %return
return: ; preds = %bb39
ret void
}