aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNadav Rotem <nrotem@apple.com>2012-09-11 12:34:27 +0000
committerNadav Rotem <nrotem@apple.com>2012-09-11 12:34:27 +0000
commit8754bbbe672450689a5bdc8a198af90144e56f31 (patch)
tree873746e9c2c6fd1ba372c238879ee73878f5eadc
parent468a95fb045dbdc4c67ce3368399d8246bf65d9e (diff)
downloadexternal_llvm-8754bbbe672450689a5bdc8a198af90144e56f31.zip
external_llvm-8754bbbe672450689a5bdc8a198af90144e56f31.tar.gz
external_llvm-8754bbbe672450689a5bdc8a198af90144e56f31.tar.bz2
Stack Coloring: Dont crash on dbg values which use stack frames.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@163616 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--lib/CodeGen/StackColoring.cpp8
-rw-r--r--test/CodeGen/X86/StackColoring-dbg.ll30
2 files changed, 35 insertions, 3 deletions
diff --git a/lib/CodeGen/StackColoring.cpp b/lib/CodeGen/StackColoring.cpp
index dbfa4bb..6df932c 100644
--- a/lib/CodeGen/StackColoring.cpp
+++ b/lib/CodeGen/StackColoring.cpp
@@ -523,10 +523,12 @@ void StackColoring::remapInstructions(DenseMap<int, int> &SlotRemap) {
// the calculated range then it means that the alloca usage moved
// outside of the lifetime markers.
#ifndef NDEBUG
- SlotIndex Index = Indexes->getInstructionIndex(I);
- LiveInterval* Interval = Intervals[FromSlot];
- assert(Interval->find(Index) != Interval->end() &&
+ if (!I->isDebugValue()) {
+ SlotIndex Index = Indexes->getInstructionIndex(I);
+ LiveInterval* Interval = Intervals[FromSlot];
+ assert(Interval->find(Index) != Interval->end() &&
"Found instruction usage outside of live range.");
+ }
#endif
// Fix the machine instructions.
diff --git a/test/CodeGen/X86/StackColoring-dbg.ll b/test/CodeGen/X86/StackColoring-dbg.ll
new file mode 100644
index 0000000..5982544
--- /dev/null
+++ b/test/CodeGen/X86/StackColoring-dbg.ll
@@ -0,0 +1,30 @@
+; RUN: llc -mcpu=corei7 -no-stack-coloring=false < %s
+
+; Make sure that we don't crash when dbg values are used.
+
+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-n8:16:32:64-S128"
+target triple = "x86_64-apple-macosx10.8.0"
+
+declare void @llvm.dbg.declare(metadata, metadata) nounwind readnone
+
+define void @foo() nounwind uwtable ssp {
+entry:
+ %x.i = alloca i8, align 1
+ %y.i = alloca [256 x i8], align 16
+ %0 = getelementptr inbounds [256 x i8]* %y.i, i64 0, i64 0
+ br label %for.body
+
+for.body:
+ call void @llvm.lifetime.end(i64 -1, i8* %0) nounwind
+ call void @llvm.lifetime.start(i64 -1, i8* %x.i) nounwind
+ call void @llvm.dbg.declare(metadata !{i8* %x.i}, metadata !22) nounwind
+ br label %for.body
+}
+
+declare void @llvm.lifetime.start(i64, i8* nocapture) nounwind
+
+declare void @llvm.lifetime.end(i64, i8* nocapture) nounwind
+
+!16 = metadata !{i32 786468, null, metadata !"char", null, i32 0, i64 8, i64 8, i64 0, i32 0, i32 6}
+!2 = metadata !{i32 0}
+!22 = metadata !{i32 786688, metadata !2, metadata !"x", metadata !2, i32 16, metadata !16, i32 0, i32 0}