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

Adding the comment is useful so it is not overlooked. My feeling is that we are 
going out of our way to avoid adding things to the cc1 command line, and 
ultimately we should make the code simpler to understand (using 
addArchDefaults) and the command line more explicit. I appreciate the intention 
to not change cc1 though. This is good for now.

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