[PATCH] D17433: [ARM] fix initialization of PredictableSelectIsExpensive
Junmo Park via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 22 23:44:25 PST 2016
flyingforyou added a comment.
Thanks for comment James.
// Prefer likely predicted branches to selects on out-of-order cores.
PredictableSelectIsExpensive = Subtarget->isLikeA9();
I think if we believe `PredictableSelectIsExpensive ` is proper for out-of-order cores, we can set this by `Subtarget->getSchedModel().isOutOfOrder()`.
And if some OoO cores have to set this false due to some regressions or whatever reasons, then we can add some code setting false for specific cores.
More information about the llvm-commits