[PATCH] D101000: Coverage: Document how to collect a profile without a filesystem
Duncan P. N. Exon Smith via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 21 16:28:39 PDT 2021
dexonsmith marked an inline comment as done.
dexonsmith added inline comments.
================
Comment at: clang/docs/SourceBasedCodeCoverage.rst:368
+
+In C++ files, declare these as ``extern "C"``.
+
----------------
vsk wrote:
> I think we should leave a note about how to avoid pulling in unwanted symbol references, and perhaps reference the preceding section about using the runtime without its static initializer.
Improved this a bit; let me know if you suggested text that makes the unwanted symbol reference stuff a bit more clear.
Note, BTW, that I've made this a sub-section of "Using the profiling runtime without static initializers"; if you think it should be a peer section that's an option too.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101000/new/
https://reviews.llvm.org/D101000
More information about the cfe-commits
mailing list