================
@@ -2507,6 +2507,7 @@ static void CollectArgsForIntegratedAssembler(Compilation 
&C,
   bool Crel = false, ExperimentalCrel = false;
   bool UseRelaxRelocations = C.getDefaultToolChain().useRelaxRelocations();
   bool UseNoExecStack = false;
+  bool Msa = false;
----------------
yingopq wrote:

> `bool Msa, NoMsa` encodes 4 states where `Msa&&NoMsa` is meaningless but not 
> checked.
> 
> `std::optional<bool> Msa` encodes the desired 3 states:
> 
> * not set
> * set and true
> * set and false

Sorry, I didn't realize you were asking me to use an option type bool MSA on 
the first version of the modification.

https://github.com/llvm/llvm-project/pull/99615
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to