vsk added inline comments.
================
Comment at: clang/lib/CodeGen/CoverageMappingGen.cpp:1393
+ size_t CoverageMappingSize = 0;
+ for (auto &S : CoverageMappings) {
+ CoverageMappingSize += S.size();
----------------
It doesn't look like the CoverageMappings std::vector is needed at all.
Consider moving `FilenamesAndCoverageMappings` into CoverageMappingModuleGen?
That should reduce the memory requirements a bit more.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62623/new/
https://reviews.llvm.org/D62623
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
