[PATCH] D67283: [GCOV] Skip artificial functions from being emitted

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 12 08:00:10 PST 2019


aganea added a comment.

@dblaikie : I had to revert *all* the revisions you're mentioning, because of this issue with GCOV instrumentation. This current patch is required before re-landing D66328 <https://reviews.llvm.org/D66328>.

For a repro with latest git upstream, you need to compile with `clang` and `LLVM_ENABLE_ASSERTIONS=ON`, and run the test `cfe/trunk/test/CodeGenCXX/debug-info-atexit-stub.cpp` from D66328 <https://reviews.llvm.org/D66328>. The issue is still there.

GCOV trips with D66328 <https://reviews.llvm.org/D66328> applied because the entry block for ``dynamic initializer for 'XXX'` is now marked as artificial and doesn't have a line #. Please see my comment <https://reviews.llvm.org/D67283#1732809> above. Let me know if it's unclear and needs more explanation.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67283/new/

https://reviews.llvm.org/D67283





More information about the llvm-commits mailing list