This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Fix intermittent failure with instrprof-gcov-multithread_fork.test on 32-bit arm
AbandonedPublic

Authored by rovka on Jun 15 2020, 2:39 AM.

Details

Reviewers
calixte
Summary

The test fails intermittently with 'terminate called recursively' errors likely
caused by having too many threads. This patch addresses the issue by
catching any exceptions thrown during thread creation.

Diff Detail

Event Timeline

rovka created this revision.Jun 15 2020, 2:39 AM
rovka added a comment.Jun 15 2020, 2:40 AM

@calixte: Does this affect the intention of the test in any way?

The base patch is not the correct one:
https://reviews.llvm.org/D81650
Oh I think I understand: the patch I made has been uplifted in branch 10 (https://bugs.llvm.org/show_bug.cgi?id=44792#c22) but no the test fix: https://reviews.llvm.org/D79621
So you must uplift patch in D79621 in branch 10 and if the issue is still there then add the try...catch like in D81650.
If it isn't clear enough then you can reach me on irc or discord.

rovka abandoned this revision.Jun 15 2020, 8:52 AM

Note, these files have nothing to do with instrprof, so they should be named gcov-*, rather than instrprof-gcov-*