[compiler-rt] r337432 - Remove scheduling dependency from XRay :: Posix/fork_basic_logging.cc

Matthew Voss via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 18 17:25:00 PDT 2018


Author: ormris
Date: Wed Jul 18 17:25:00 2018
New Revision: 337432

URL: http://llvm.org/viewvc/llvm-project?rev=337432&view=rev
Log:
Remove scheduling dependency from XRay :: Posix/fork_basic_logging.cc

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

Modified:
    compiler-rt/trunk/test/xray/TestCases/Posix/fork_basic_logging.cc

Modified: compiler-rt/trunk/test/xray/TestCases/Posix/fork_basic_logging.cc
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/xray/TestCases/Posix/fork_basic_logging.cc?rev=337432&r1=337431&r2=337432&view=diff
==============================================================================
--- compiler-rt/trunk/test/xray/TestCases/Posix/fork_basic_logging.cc (original)
+++ compiler-rt/trunk/test/xray/TestCases/Posix/fork_basic_logging.cc Wed Jul 18 17:25:00 2018
@@ -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;
 }




More information about the llvm-commits mailing list