[PATCH] D17433: [ARM] fix initialization of PredictableSelectIsExpensive; NFC
Renato Golin via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 21 04:38:09 PST 2016
rengolin added reviewers: t.p.northover, rengolin.
rengolin added a comment.
I think this change does make sense, but I'll let James or Tim have a look, since this needs more than a pair of eyes.
Have you made any benchmarks for this? On both A9 and A15 as well as AArch32? Without concrete numbers, it's easy to assume the best and get the worst.
cheers,
--renato
http://reviews.llvm.org/D17433
More information about the llvm-commits
mailing list