[PATCH] D20453: Remove optnone/opt-bisect check from StackColoring

[Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL271068: Update the stack coloring pass to remove lifetime intrinsics in the… (authored by akaylor).

Changed prior to commit:
  http://reviews.llvm.org/D20453?vs=58721&id=58859#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D20453

Files:
  llvm/trunk/lib/CodeGen/StackColoring.cpp
  llvm/trunk/test/Feature/optnone-llc.ll
  llvm/trunk/test/Other/opt-bisect-legacy-pass-manager.ll

Index: llvm/trunk/test/Feature/optnone-llc.ll
===================================================================
--- llvm/trunk/test/Feature/optnone-llc.ll
+++ llvm/trunk/test/Feature/optnone-llc.ll
@@ -46,7 +46,6 @@
 ; LLC-Ox-DAG: Skipping pass 'Machine Copy Propagation Pass'
 ; LLC-Ox-DAG: Skipping pass 'Machine Instruction Scheduler'
 ; LLC-Ox-DAG: Skipping pass 'Machine Loop Invariant Code Motion'
-; LLC-Ox-DAG: Skipping pass 'Merge disjoint stack slots'
 ; LLC-Ox-DAG: Skipping pass 'Optimize machine instruction PHIs'
 ; LLC-Ox-DAG: Skipping pass 'Peephole Optimizations'
 ; LLC-Ox-DAG: Skipping pass 'Post{{.*}}RA{{.*}}{{[Ss]}}cheduler'
Index: llvm/trunk/test/Other/opt-bisect-legacy-pass-manager.ll
===================================================================
--- llvm/trunk/test/Other/opt-bisect-legacy-pass-manager.ll
+++ llvm/trunk/test/Other/opt-bisect-legacy-pass-manager.ll
@@ -154,3 +154,25 @@
 bb.false:
   ret i32 0
 }
+
+; This function is here to verify that opt-bisect can skip all passes for
+; functions that contain lifetime intrinsics.
+define void @f4() {
+entry:
+  %i = alloca i32, align 4
+  %tmp = bitcast i32* %i to i8*
+  call void @llvm.lifetime.start(i64 4, i8* %tmp)
+  br label %for.cond
+
+for.cond:
+  br i1 undef, label %for.body, label %for.end
+
+for.body:
+  br label %for.cond
+
+for.end:
+  ret void
+}
+
+declare void @llvm.lifetime.start(i64, i8* nocapture)
+
Index: llvm/trunk/lib/CodeGen/StackColoring.cpp
===================================================================
--- llvm/trunk/lib/CodeGen/StackColoring.cpp
+++ llvm/trunk/lib/CodeGen/StackColoring.cpp
@@ -963,9 +963,6 @@
 }
 
 bool StackColoring::runOnMachineFunction(MachineFunction &Func) {
-  if (skipFunction(*Func.getFunction()))
-    return false;
-
   DEBUG(dbgs() << "********** Stack Coloring **********\n"
                << "********** Function: "
                << ((const Value*)Func.getFunction())->getName() << '\n');
@@ -1005,7 +1002,8 @@
 
   // Don't continue because there are not enough lifetime markers, or the
   // stack is too small, or we are told not to optimize the slots.
-  if (NumMarkers < 2 || TotalSize < 16 || DisableColoring) {
+  if (NumMarkers < 2 || TotalSize < 16 || DisableColoring ||
+      skipFunction(*Func.getFunction())) {
     DEBUG(dbgs()<<"Will not try to merge slots.\n");
     return removeAllMarkers();
   }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20453.58859.patch
Type: text/x-patch
Size: 2382 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160527/57d21543/attachment.bin>