[PATCH] D64415: Consistent types and naming for AArch64 target features (NFC)
Peter Smith via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 15 02:10:24 PDT 2019
peter.smith added a comment.
FWIW I think this looks reasonable. The ARM equivalent uses bitfields such as
unsigned CRC : 1;
unsigned Crypto : 1;
unsigned DSP : 1;
unsigned Unaligned : 1;
unsigned DotProd : 1;
Which would make more sense than using unsigned for each individual field. Several other targets that I looked at used bool and the Has<feature> convention so I think this brings AArch64 more in line with non ARM Targets at the cost of losing a little coherence with ARM where the name Crypto remains. I don't think that this is particularly important as the two have already started to diverge with HasDotProd.
CHANGES SINCE LAST ACTION
More information about the cfe-commits