[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>();
     AU.addRequiredID(LowerSwitchID);
----------------
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 


http://reviews.llvm.org/D16602





More information about the llvm-commits mailing list