[PATCH] D72033: [AArch64] Enable post-RA sched for more cores

Tue Dec 31 06:46:46 PST 2019

samparker marked an inline comment as done.
samparker added a comment.

> But I am not sure what the tests check exactly. Wouldn’t it be sufficient to check if the post ra scheduler is run per default for the additional cores?

Yes, this is all I really what to test... what's the easiest way to do this?

================
Comment at: llvm/lib/Target/AArch64/AArch64.td:542
                                    FeatureFuseAES,
+                                   FeatureFuseLiterals,
                                    FeatureNEON,
----------------
fhahn wrote:
> This is unrelated to the postra scheduling, right? It would be slightly better to split it off and have it as separate change, with tests just for the feature
Ok.

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

https://reviews.llvm.org/D72033