[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