[PATCH] D17433: [ARM] fix initialization of PredictableSelectIsExpensive; NFC

Junmo Park via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 21 17:31:34 PST 2016


flyingforyou added a comment.

Thanks for detailed comment, Sanjay.

As you said, 
`CortexA9Model `: cortex-a9, a12, a15, a17, krait
`SwiftModel `: swift, cyclone
`isLikeA9 `: cortex-a9,a15,krait

above targets will be affected by "PredictableSelectIsExpensive".

Hi, renato.

> Junmo, please remove the NFC comment from the commit message.


Sure, I will.

I don't have access to a9/a15/a17. (What I have is only for cortex-a53/57, x86's i series)
And I also don't have access to swift/cyclone. I have iphone devices. But I don't know how to measure the performance with test-suite.

Thanks.


http://reviews.llvm.org/D17433





More information about the llvm-commits mailing list