[PATCH] D52033: [GCOV] Add options to filter files which must be instrumented.

calixte via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 26 00:00:49 PDT 2018


calixte added inline comments.


================
Comment at: include/llvm/Transforms/Instrumentation.h:69
+  // Regex separated by a comma to filter the files to instrument.
+  std::string Filter;
+
----------------
marco-c wrote:
> Nit: maybe you could give a better name to these options, e.g. something like `FilterFiles` and `ExcludeFiles`.
The two variables here have the same name as the options (ie coverage-filter & coverage-exclude)


Repository:
  rL LLVM

https://reviews.llvm.org/D52033





More information about the llvm-commits mailing list