[clang] [llvm] [AArch64] Decouple feature dependency expansion. (PR #94279)

Tomas Matheson via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 4 10:02:25 PDT 2024


================
@@ -166,6 +174,11 @@ void AArch64::ExtensionSet::enable(ArchExtKind E) {
   Touched.set(E);
   Enabled.set(E);
 
+  // These depend on each other, meaning you can't have one without the other.
+  // We don't want this to be in ExtensionDependencies to avoid infinite loops.
+  if (E == AEK_FP)
+    enable(AEK_SIMD);
----------------
tmatheson-arm wrote:

Unrelated change? They do in the Arm ARM, but we want them to be independently configurable in LLVM.

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


More information about the llvm-commits mailing list