mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-08-10 02:29:32 +00:00
6c7a7c6474
The symptom is that an assertion is triggered. The assertion was added by
me to detect the situation when value is propagated from dead blocks.
(We can certainly get rid of assertion; it is safe to do so, because propagating
value from dead block to alive join node is certainly ok.)
The root cause of this bug is : edge-splitting is conducted on the fly,
the edge being split could be a dead edge, therefore the block that
split the critial edge needs to be flagged "dead" as well.
There are 3 ways to fix this bug:
1) Get rid of the assertion as I mentioned eariler
2) When an dead edge is split, flag the inserted block "dead".
3) proactively split the critical edges connecting dead and live blocks when
new dead blocks are revealed.
This fix go for 3) with additional 2 LOC.
Testing case was added by Rafael the other day.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@194424 91177308-0d34-0410-b5e6-96231b3b80d8
41 lines
1.0 KiB
LLVM
41 lines
1.0 KiB
LLVM
; RUN: opt < %s -basicaa -gvn -S | FileCheck %s
|
|
|
|
@last = external global [65 x i32*]
|
|
|
|
define i32 @NextRootMove(i32 %wtm, i32 %x, i32 %y, i32 %z) {
|
|
entry:
|
|
%A = alloca i32*
|
|
%tmp17618 = load i32** getelementptr ([65 x i32*]* @last, i32 0, i32 1), align 4
|
|
store i32* %tmp17618, i32** %A
|
|
; CHECK: entry:
|
|
; CHECK-NEXT: alloca i32
|
|
; CHECK-NEXT: %tmp17618 = load
|
|
; CHECK-NOT: load
|
|
; CHECK-NOT: phi
|
|
br label %cond_true116
|
|
|
|
cond_true116:
|
|
%cmp = icmp eq i32 %x, %y
|
|
br i1 %cmp, label %cond_true128, label %cond_true145
|
|
|
|
cond_true128:
|
|
%tmp17625 = load i32** getelementptr ([65 x i32*]* @last, i32 0, i32 1), align 4
|
|
store i32* %tmp17625, i32** %A
|
|
%cmp1 = icmp eq i32 %x, %z
|
|
br i1 %cmp1 , label %bb98.backedge, label %return.loopexit
|
|
|
|
bb98.backedge:
|
|
br label %cond_true116
|
|
|
|
cond_true145:
|
|
%tmp17631 = load i32** getelementptr ([65 x i32*]* @last, i32 0, i32 1), align 4
|
|
store i32* %tmp17631, i32** %A
|
|
br i1 false, label %bb98.backedge, label %return.loopexit
|
|
|
|
return.loopexit:
|
|
br label %return
|
|
|
|
return:
|
|
ret i32 0
|
|
}
|