[clang] [APINotes] Upstream Driver and Frontend options that enable API Notes (PR #73120)
Saleem Abdulrasool via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 23 08:43:30 PST 2023
================
@@ -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
More information about the cfe-commits
mailing list