Skip to content

Commit

Permalink
Remove scheduling dependency from XRay :: Posix/fork_basic_logging.cc
Browse files Browse the repository at this point in the history
Summary:
We've been seeing intermittent failures on our internal bots and we suspect
this may be due to the OS scheduling the child process to run before the parent
process.

This version ensures that the parent and child can be run in either order.

Reviewers: Maknee, dberris

Reviewed By: dberris

Subscribers: delcypher, #sanitizers, Maknee, llvm-commits

Differential Revision: https://reviews.llvm.org/D49501

llvm-svn: 337432
  • Loading branch information
ormris committed Jul 19, 2018
1 parent 53c30ed commit ac1f926
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions compiler-rt/test/xray/TestCases/Posix/fork_basic_logging.cc
Expand Up @@ -70,12 +70,12 @@ int main()
if(fork())
{
print_parent_or_child();
// CHECK: Parent with tid
// CHECK-DAG: Parent with tid
}
else
{
print_parent_or_child();
// CHECK: Child with tid
// CHECK-DAG: Child with tid
}
return 0;
}
Expand Down

0 comments on commit ac1f926

Please sign in to comment.