================
@@ -340,14 +340,22 @@ void AArch64::ExtensionSet::disable(ArchExtKind E) {
       disable(Dep.Later);
 }
 
+static void setEnableIfMandatory(AArch64::ExtensionSet &Exts,
----------------
tommat01 wrote:

There aren't really any "mandatory" extensions as far as the compiler is 
concerned; you are free to avoid code generation for features even if they are 
required to be implemented in the CPU.

I'd probably inline this function and use the existing 
`ExtensionSet::addArchDefaults`.

https://github.com/llvm/llvm-project/pull/203458
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to