[PATCH] D52256: AArch64: Add FuseCryptoApple fusion rules

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 19 01:44:52 PDT 2018


javed.absar added inline comments.


================
Comment at: lib/Target/AArch64/AArch64.td:159
 
+def FeatureFuseCryptoApple : SubtargetFeature<
+    "fuse-crypto", "HasFuseCryptoApple", "true",
----------------
Would it be better to give a more operational name (e.g. FuseAESCrypto?) along with a comment if it applies to only certain configs. Just a thought, I am ok with as it is as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D52256





More information about the llvm-commits mailing list