[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


================
@@ -861,12 +861,10 @@ bool AArch64TargetInfo::handleTargetFeatures(std::vector<std::string> &Features,
       HasSVE2 = true;
       HasSVE2p1 = true;
     }
-    if (Feature == "+sve2-aes") {
+    if (Feature == "+sve-aes") {
----------------
labrinea wrote:

If I understand correctly the TargetParser (invoked from the driver or the target attribute parser) will expand the dependencies of `sve2-aes` to {sve-aes, sve2} which are handled here separately. I guess that works so perhaps it's not necessary to preserve the code you are removing. At first I was worried.

https://github.com/llvm/llvm-project/pull/116026


More information about the cfe-commits mailing list