[PATCH] D91848: [AArch64] Enable post RA scheduler for Cortex-R82

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 20 02:58:07 PST 2020


dmgreen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64.td:674
+                               FeaturePostRAScheduler,
+                               FeatureCrypto,
+                               FeatureFuseAES,
----------------
If you are adding crypto (which doesn't sound like a bad idea to me), should there be some other tests for that?

If you want to keep this simple and do that in a separate patch, you could just add -mattr=+crypto to the r82 run line.


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

https://reviews.llvm.org/D91848



More information about the llvm-commits mailing list