[PATCH] D89359: [llvm-cov] don't include all source files when provided source files are filtered out

Max Moroz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 9 14:18:50 PST 2020


Dor1s added inline comments.


================
Comment at: llvm/tools/llvm-cov/CodeCoverage.cpp:414
     sys::path::native(Filename, NativeFilename);
+    sys::path::remove_dots(NativeFilename, true);
     if (NativeFilename.startswith(RemapFrom)) {
----------------
zequanwu wrote:
> Dor1s wrote:
> > Why was this line added? Looks like it broke some big deployments: https://github.com/google/oss-fuzz/issues/4608
> > 
> > 
> I put a fix here: https://github.com/llvm/llvm-project/commit/c75a0a1e9dc29be4e00d37d0d00288afc1a6153f
Thank you for the quick response!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89359



More information about the llvm-commits mailing list