[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