This is an archive of the discontinued LLVM Phabricator instance.

tsan: deflake a test
ClosedPublic

Authored by dvyukov on Jan 29 2018, 12:45 AM.

Details

Reviewers
kubamracek
Summary

There was a failure on a bot:
http://lab.llvm.org:8011/builders/clang-cmake-mipsel/builds/1283

strerror test is indeed flaky. We protect all races by a barrier in other tests
to eliminate flakiness. Do the same here.

No idea why tls_race2.cc failed. Add output at the end of the test
as we do in other tests. Sometimes test process crashes somewhere
in the middle (e.g. during race reporting) and it looks like empty output.
Output at the end of test allows to understand if the process has crashed,
or it has finished but produced no race reports.

Diff Detail

Event Timeline

dvyukov created this revision.Jan 29 2018, 12:45 AM
kubamracek accepted this revision.Jan 29 2018, 7:08 AM
This revision is now accepted and ready to land.Jan 29 2018, 7:08 AM
dvyukov closed this revision.Jan 29 2018, 7:12 AM

Submitted in r323657.