[PATCH] D148757: [clang] Apply -fcoverage-prefix-map reverse order
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 26 16:44:51 PDT 2023
MaskRay added inline comments.
================
Comment at: clang/include/clang/Basic/CodeGenOptions.h:211
+
+ /// Prefix replacement map for coverage.
+ llvm::SmallVector<std::pair<std::string, std::string>, 0> CoveragePrefixMap;
----------------
MaskRay wrote:
> While adding a comment, clarify what coverage it is? There are multiple coverage instrumentation features in Clang.
The declaration should not repeat the comment at the use site.
I am asking for what coverage features this CoveragePrefixMap is used for...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148757/new/
https://reviews.llvm.org/D148757
More information about the cfe-commits
mailing list