[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