This is an archive of the discontinued LLVM Phabricator instance.

Flush gcda files before unlocking them

Authored by marco-c on Nov 29 2017, 9:15 AM.

Diff Detail

Event Timeline

marco-c created this revision.Nov 29 2017, 9:15 AM
Herald added subscribers: Restricted Project, llvm-commits. · View Herald TranscriptNov 29 2017, 9:15 AM

I added two more reviewers as we would like to get that in trunk before 6.0 branches.

FYI, this was necessary to fix some intermittent issues for Firefox code coverage.

marco-c updated this revision to Diff 128511.Jan 3 2018, 3:13 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jan 3 2018, 3:14 AM
This revision was automatically updated to reflect the committed changes.

The change is very contained and safe and we need it in 6.0.