[PATCH] D25482: [PPC] Allow two post RA schedulers to be in the pipeline and select one depending on the Machine Function's subtarget

[Ehsan Amiri via llvm-commits]

amehsan added a comment.

>   you add a generic mechanism that affects every pass calling skipFunction().

This patch has zero impact on any pass other than the two PostRA schedulers.

>   I don't see how that is worse than adding one more if() to every MachinePass. 

Why do you think this patch adds one more `if()` to every MachinePass? The impact on any pass other than the two PostRA patches, is literally zero.


https://reviews.llvm.org/D25482