[PATCH] D95753: Store compilation dir separately in coverage mapping

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 15 15:20:22 PST 2021


MaskRay 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]>,
----------------
Using fcoverage_compilation_dir_EQ for both driver option and CC1 option is preferred.

Can you drop fcoverage_compilation_dir ?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95753/new/

https://reviews.llvm.org/D95753



More information about the cfe-commits mailing list