This is an archive of the discontinued LLVM Phabricator instance.

[profile] Fix the tests for patch in

Authored by calixte on Jul 27 2018, 7:25 AM.



The goal of D49916 is to remove some hit counters from lines like '}'.
So as a consequence, the tests must be fixed.
FYI, after holidays, I'll add more tests.

Event Timeline

calixte created this revision.Jul 27 2018, 7:25 AM
Herald added subscribers: Restricted Project, llvm-commits, delcypher. · View Herald TranscriptJul 27 2018, 7:25 AM
calixte edited the summary of this revision. (Show Details)Jul 27 2018, 7:26 AM
marco-c added inline comments.Sep 20 2018, 2:40 AM

Shouldn't this be 1?

calixte added inline comments.Sep 21 2018, 1:58 AM

With the patch to attach function line def to first block, we'll have 1. But here we've 0 since "ret void" isn't instrumented.

calixte updated this revision to Diff 166413.Sep 21 2018, 1:58 AM

Fix the tests.

marco-c accepted this revision.Sep 21 2018, 2:11 AM
This revision is now accepted and ready to land.Sep 21 2018, 2:11 AM
This revision was automatically updated to reflect the committed changes.