================
@@ -2168,6 +2168,15 @@ void Clang::AddSystemZTargetArgs(const ArgList &Args,
     CmdArgs.push_back("-mfloat-abi");
     CmdArgs.push_back("soft");
   }
+
+  if (Arg *A = Args.getLastArg(options::OPT_mzos_ppa1_name,
----------------
perry-ca wrote:

Move this code inside of the `if (Triple.isOSzOS())` block (flip the order of 
the if statements) and remove the code that prints the error message.  Marking 
the option as target specific and only calling getLastArg() on z/OS will end 
using the existing mechanism for generating that error.

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

Reply via email to