[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