[PATCH] D46884: [AArch64] Cortex-A55 scheduler model

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

In D46884#2284640 <https://reviews.llvm.org/D46884#2284640>, @SjoerdMeijer wrote:

> @evgeny777 : how about we commit this to have a baseline that we iterate on?

I feel like that could cause problems in the short term, if the information in it is incorrect and it degraded performance. We could perhaps commit it without enabling it to begin with.

I would like to see the tests being improved too. Something like the equivalent of llvm/test/tools/llvm-mca/ARM/m4-int.s would be very useful for checking the values are all present and what we expect.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D46884/new/

https://reviews.llvm.org/D46884