[PATCH] D16602: StructurizeCFG: Add an option for skipping regions with only uniform branches

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 20:39:30 PST 2016

arsenm added inline comments.

Comment at: lib/Transforms/Scalar/StructurizeCFG.cpp:264
@@ -252,2 +263,3 @@
   void getAnalysisUsage(AnalysisUsage &AU) const override {
+    AU.addRequired<DivergenceAnalysis>();
This should not be a required analysis. Even with the false option, it will force running the analysis. I think if the target pass config explicitly adds the analysis pass before StructurizeCFG it should work 


More information about the llvm-commits mailing list