[clang] [llvm] Make +sve2-aes an alias of +sve2+sve-aes (PR #116026)
Alexandros Lamprineas via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 13 07:18:59 PST 2024
================
@@ -769,7 +769,7 @@ bool AArch64TargetInfo::hasFeature(StringRef Feature) const {
.Case("f32mm", FPU & SveMode && HasMatmulFP32)
.Case("f64mm", FPU & SveMode && HasMatmulFP64)
.Case("sve2", FPU & SveMode && HasSVE2)
- .Case("sve2-pmull128", FPU & SveMode && HasSVE2AES)
+ .Case("sve-aes", HasSVEAES)
----------------
labrinea wrote:
The function AArch64TargetInfo::hasFeature doesn't seem to be used much. I don't see any particular tests for it neither it is clear whether the stringswitch should be handling backend feature names or command line facing feature names. I bet most of the cases are useless. That said it doesn't seem necessary to add a line for `.Case("sve2-aes", HasSVE2 && HasSVEAES)` I guess.
https://github.com/llvm/llvm-project/pull/116026
More information about the cfe-commits
mailing list