[PATCH] D120906: [ARM][AArch64] generate subtarget feature flags [NFC]

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 11 18:38:52 PST 2022


MaskRay added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARM.td:191
 
+// If true, processor supports AES
 def FeatureAES            : SubtargetFeature<"aes", "HasAES", "true",
----------------
`supports AES` does not convey more information than `FeatureAES`. Drop the comment.

(The previous comment `// If true, processor supports SHA1 and SHA256` conveys more information.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120906



More information about the llvm-commits mailing list