[ORC][RPC] Join server thread before checking condition in unit test.

Otherwise we have a race on the sent-messages count.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@371263 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/unittests/ExecutionEngine/Orc/RPCUtilsTest.cpp b/unittests/ExecutionEngine/Orc/RPCUtilsTest.cpp
index 8e4c5330d90ecaa2f3b20565d7259031da990d25..2a0ed07725241d4fdec5b32266e2ddf4194e708e 100644 (file)
--- a/unittests/ExecutionEngine/Orc/RPCUtilsTest.cpp
+++ b/unittests/ExecutionEngine/Orc/RPCUtilsTest.cpp
@@ -214,6 +214,8 @@ TEST(DummyRPC, TestCallAsyncVoidBool) {
     EXPECT_FALSE(!!Err) << "Client failed to handle response from void(bool)";
   }
 
+  ServerThread.join();
+
   // The client should have made two calls to send: One implicit call to
   // negotiate the VoidBool function key, and a second to make the VoidBool
   // call.
@@ -224,8 +226,6 @@ TEST(DummyRPC, TestCallAsyncVoidBool) {
   // the negotiate call, and another to send the response to the VoidBool call.
   EXPECT_EQ(Channels.second->SendCalls, 2U)
       << "Expected two send calls to have been made by server";
-
-  ServerThread.join();
 }
 
 TEST(DummyRPC, TestCallAsyncIntInt) {