[PATCH] D13044: AMDGPU: Disable some passes that are not meaningful

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 25 10:43:29 PDT 2015


arsenm marked an inline comment as done.

================
Comment at: lib/Target/AMDGPU/AMDGPUTargetMachine.cpp:300-303
@@ -293,1 +299,6 @@
+
+  // Exceptions and StackMaps are not supported, so these passes will never do
+  // anything.
+  disablePass(&StackMapLivenessID);
+  disablePass(&FuncletLayoutID);
 }
----------------
tstellarAMD wrote:
> Would it be better to disable the passes from the GCNPassConfig constructor?
> 
> LGTM.
> 
> 
I moved it to the AMDGPUPassConfig constructor


http://reviews.llvm.org/D13044





More information about the llvm-commits mailing list