================
@@ -6720,6 +6720,17 @@ void Clang::ConstructJob(Compilation &C, const JobAction 
&JA,
   Args.addOptOutFlag(CmdArgs, options::OPT_fassume_sane_operator_new,
                      options::OPT_fno_assume_sane_operator_new);
 
+  if (Args.hasFlag(options::OPT_fapinotes, options::OPT_fno_apinotes, false) ||
+      Args.hasArg(options::OPT_iapinotes_modules)) {
----------------
compnerd wrote:

Ah, now with the new version, it appears the double check is to deal with 
`-fapinotes-swift-version`?  I wonder if we can simplify this at all.  Is it 
invalid to pass along `-fapinotes-swift-version` if `-fapinotes` or 
`-fapinotes-modules` is not present?

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

Reply via email to