[PATCH] D148757: [clang] Apply -fcoverage-prefix-map reverse order
Gulfem Savrun Yeniceri via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 25 17:12:20 PDT 2023
gulfem marked 2 inline comments as done.
gulfem added inline comments.
================
Comment at: clang/lib/CodeGen/CoverageMappingGen.cpp:1638
: CGM(CGM), SourceInfo(SourceInfo) {
- CoveragePrefixMap = CGM.getCodeGenOpts().CoveragePrefixMap;
+ for (const auto &[From, To] : CGM.getCodeGenOpts().CoveragePrefixMap)
+ CoveragePrefixMap.emplace_back(From, To);
----------------
MaskRay wrote:
> `CoveragePrefixMap` can be removed. I just removed `DebugPrefixMap` as well:)
I'm marking this `Done` as you removed it in https://github.com/llvm/llvm-project/commit/d3e121780ac2724969a030415f3092170f3c7589.
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