gulfem 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: > 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... @MaskRay does this sound good to you? ``` /// Prefix replacement map for source-based code coverage to remap source /// file paths in coverage mapping. llvm::SmallVector<std::pair<std::string, std::string>, 0> CoveragePrefixMap; ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D148757/new/ https://reviews.llvm.org/D148757 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
