6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-02-08 21:32:39 +00:00
Code Issues Projects Releases Wiki Activity

Fix a crash when threading a block that includes a MRV call result.

Browse Source 
DemoteRegToStack doesn't work with MRVs yet, because it relies on the
ability to load/store things.

This fixes PR2285.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@50667 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2008-05-05 20:21:22 +00:00
parent be5918c079
commit 8554cc2ea7
2 changed files with 44 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
lib/Transforms/Scalar/JumpThreading.cpp
View File

@ -433,12 +433,27 @@ void JumpThreading::ThreadEdge(BasicBlock *BB, BasicBlock *PredBB,
// Jump Threading can not update SSA properties correctly if the values // Jump Threading can not update SSA properties correctly if the values
// defined in the duplicated block are used outside of the block itself. For // defined in the duplicated block are used outside of the block itself. For
// this reason, we spill all values that are used outside of BB to the stack. // this reason, we spill all values that are used outside of BB to the stack.
for (BasicBlock::iterator I = BB->begin(); I != BB->end(); ++I) for (BasicBlock::iterator I = BB->begin(); I != BB->end(); ++I) {
if (I->isUsedOutsideOfBlock(BB)) { if (!I->isUsedOutsideOfBlock(BB))
// We found a use of I outside of BB. Create a new stack slot to continue;
// break this inter-block usage pattern.
// We found a use of I outside of BB. Create a new stack slot to
// break this inter-block usage pattern.
if (!isa<StructType>(I->getType())) {
DemoteRegToStack(*I); DemoteRegToStack(*I);
continue;
} }
// Alternatively, I must be a call or invoke that returns multiple retvals.
// We can't use 'DemoteRegToStack' because the at will create loads and
// stores of aggregates which is not valid yet. If I is a call, we can just
// pull all the getresult instructions up to this block. If I is an invoke,
// we are out of luck.
BasicBlock::iterator IP = I; ++IP;
for (Value::use_iterator UI = I->use_begin(), E = I->use_end();
UI != E; ++UI)
cast<GetResultInst>(UI)->moveBefore(IP);
}
// We are going to have to map operands from the original BB block to the new // We are going to have to map operands from the original BB block to the new
// copy of the block 'NewBB'. If there are PHI nodes in BB, evaluate them to // copy of the block 'NewBB'. If there are PHI nodes in BB, evaluate them to

25
test/Transforms/JumpThreading/2008-05-05-MRV-Crash.ll Normal file
View File

@ -0,0 +1,25 @@
; RUN: llvm-as < %s | opt -jump-threading -disable-output
; PR2285
target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128"
target triple = "x86_64-unknown-linux-gnu"
%struct.system__secondary_stack__mark_id = type { i64, i64 }
define void @_ada_c35507b() {
entry:
br label %bb
bb: ; preds = %bb13, %entry
%ch.0 = phi i8 [ 0, %entry ], [ 0, %bb13 ] ; <i8> [#uses=1]
%tmp11 = icmp ugt i8 %ch.0, 31 ; <i1> [#uses=1]
%tmp120 = call %struct.system__secondary_stack__mark_id @system__secondary_stack__ss_mark( ) ; <%struct.system__secondary_stack__mark_id> [#uses=1]
br i1 %tmp11, label %bb110, label %bb13
bb13: ; preds = %bb
br label %bb
bb110: ; preds = %bb
%mrv_gr124 = getresult %struct.system__secondary_stack__mark_id %tmp120, 1 ; <i64> [#uses=0]
unreachable
}
declare %struct.system__secondary_stack__mark_id @system__secondary_stack__ss_mark()