================ @@ -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