[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