SjoerdMeijer updated this revision to Diff 104380.
SjoerdMeijer added a comment.
Thanks! I am now using llvm::AArch64::ArchKind. And I agree that the check for
setting __ARM_FEATURE_QRDMX is suspicious. I will address this separately.
https://reviews.llvm.org/D34686
Files:
lib/Basic/Targets.
rogfer01 added a comment.
I agree with the spirit of the change but I think you want to use `AArch64::`
enumerators instead of those in `ARM::`
Comment at: lib/Basic/Targets.cpp:6190
unsigned HasFullFP16;
+ unsigned ArchKind;
There is an `AArch64::ArchKi
SjoerdMeijer created this revision.
Herald added subscribers: kristof.beyls, aemerson.
This is a clean-up for different ARMV8-A architecture kinds. Helper function
hasFP16VectorArithmetic makes things a bit more “scalable” if we want to add
ARMv8.3 at some point.
https://reviews.llvm.org/D3468
