[RGT] RPCUtilsTest, replace un-executed EXPECT with unreachable

[probinson]

Summary

Unreachable code should be self-documented as unreachable.

Found by the Rotten Green Tests project.

[lhames]

LGTM. Thanks Paul!

(If you don't have commit access already just let me know and I can land this on your behalf)

llvm/unittests/ExecutionEngine/Orc/RPCUtilsTest.cpp

TEST(DummyRPC, TestAPICalls) {
 
   ServerThread.join();
 }
 
 TEST(DummyRPC, TestRemoveHandler) {
   auto Channels = createPairedQueueChannels();
   DummyRPCEndpoint Server(*Channels.second);
 
   Server.addHandler<DummyRPCAPI::VoidBool>(
     [](bool B) {
-      EXPECT_EQ(B, true)
-        << "Server void(bool) received unexpected result";
+      llvm_unreachable("Server void(bool) received unexpected result");
     });
 
   Server.removeHandler<DummyRPCAPI::VoidBool>();
 }
 
 TEST(DummyRPC, TestClearHandlers) {
   auto Channels = createPairedQueueChannels();
   DummyRPCEndpoint Server(*Channels.second);
 
   Server.addHandler<DummyRPCAPI::VoidBool>(
     [](bool B) {
-      EXPECT_EQ(B, true)
-        << "Server void(bool) received unexpected result";
+      llvm_unreachable("Server void(bool) received unexpected result");
     });
 
   Server.clearHandlers();
 }