[PATCH] D13708: AArch64: Disable the latency heuristic

[James Molloy via llvm-commits]

jmolloy added a comment.

Hi Matthias,

Sorry for taking so long to spot this on my backlog.

I've just run a bunch of benchmarking, and I can fairly conclusively say that your patch reduces performance overall on both Cortex-A57 and Cortex-A53 (although with a mutually exclusive set of benchmarks, which is abnormal).

I see a 43% regression in lnt.MultiSource/Benchmarks/Trimaran/enc-pc1/enc-pc1 on Cortex-A57, and a 15% regression on lnt.SingleSource/Benchmarks/Misc/salsa20 on Cortex-A53 (there are many more regressions and some improvements - these are just the top).

So I think this should indeed be gated on Cyclone.

James


Repository:
  rL LLVM

http://reviews.llvm.org/D13708