[PATCH] D83154: clang: Add -fcoverage-prefix-map

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 6 18:12:41 PDT 2020


MaskRay added a comment.

GCC obtained -fdebug-prefix-map first, then r256847 added both -fmacro-prefix-map and -ffile-prefix-map. GCC doesn't have -fcoverage-prefix-map and I saw your https://github.com/apple/swift/pull/32416
However, do we really need one new -f*-prefix-map for every feature?

If you think `-fcoverage-prefix-map=` is really necessary, you can probably ask whether GCC can adopt a -fcoverage-prefix-map for their gcov (.gcno files). If they do, reviewers may have more have more confidence to say that we should have the option as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83154





More information about the cfe-commits mailing list