[clang] [llvm] [AArch64] Add ability to list extensions enabled for a target (PR #95805)

Tomas Matheson via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 17 09:20:56 PDT 2024


================
@@ -116,7 +116,9 @@ const AArch64::ArchInfo *AArch64::parseArch(StringRef Arch) {
 std::optional<AArch64::ExtensionInfo>
 AArch64::parseArchExtension(StringRef ArchExt) {
   for (const auto &A : Extensions) {
-    if (ArchExt == A.Name || ArchExt == A.Alias)
+    if (A.UserVisibleName.empty() && !A.Alias)
+      continue;
----------------
tmatheson-arm wrote:

This seems redundant when looking at the following `if` condition, maybe we actually want to check if `ArchExt.empty()` instead?

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


More information about the cfe-commits mailing list