[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