phosek added inline comments.
================ Comment at: clang/include/clang/Driver/Options.td:1131 NegFlag<SetFalse, [], "Disable code coverage analysis">, BothFlags<[CoreOption]>>; +def fcoverage_compilation_dir : Separate<["-"], "fcoverage-compilation-dir">, + Group<f_Group>, Flags<[CC1Option, CC1AsOption, CoreOption]>, ---------------- MaskRay wrote: > Using fcoverage_compilation_dir_EQ for both driver option and CC1 option is > preferred. > > Can you drop fcoverage_compilation_dir ? I've been following `-fdebug-compilation-dir` which supports both variants. Should we drop `-fdebug-compilation-dir` as well? I'd like to keep these consistent. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95753/new/ https://reviews.llvm.org/D95753 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
