[PATCH] D20030: [AArch64] Add option to disable speculation of triangle whose tail is the only latch block

Balaram Makam via llvm-commits llvm-commits at lists.llvm.org
Mon May 16 07:31:23 PDT 2016


bmakam added a comment.

In http://reviews.llvm.org/D20030#430780, @mcrosier wrote:

> In http://reviews.llvm.org/D20030#430743, @rengolin wrote:
>
> > Hi Balaram,
> >
> > This seems like a good thing to do overall, not just for Kryo, or when the option is chosen.
>
>
> I agree and would advocate enabling this by default after some additional testing.
>
> > It would be good to know how it performs in vanilla AArch64 cores (A53, A57) so we could enable them by default.
>
>
> We should be able to get numbers for at least A57, right?


Thanks Renato and Chad,

Yes I will test it on A57 and report the results, but I need others help in testing other AArch64 targets as I do not have access to them.

> > cheers,

> 

> > --renato

> 





http://reviews.llvm.org/D20030





More information about the llvm-commits mailing list