[PATCH] D30333: Split SimplifyCFG to run obscuring switch transforms only during last phase

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: lib/Transforms/Scalar/SimplifyCFGPass.cpp:256
+  }
+};
 }
----------------
joerg wrote:
> mehdi_amini wrote:
> > Why two passes instead of a parameter?
> Two passes makes it easier to test it on the command line.
I can understand this aspect, but on the other hand that does not scale with adding more options, which is why we rarely do this. What other passes have the same approach?


https://reviews.llvm.org/D30333