diff options
| author | Dimitry Andric <dim@FreeBSD.org> | 2018-07-28 10:51:19 +0000 |
|---|---|---|
| committer | Dimitry Andric <dim@FreeBSD.org> | 2018-07-28 10:51:19 +0000 |
| commit | eb11fae6d08f479c0799db45860a98af528fa6e7 (patch) | |
| tree | 44d492a50c8c1a7eb8e2d17ea3360ec4d066f042 /unittests/ExecutionEngine/Orc/RemoteObjectLayerTest.cpp | |
| parent | b8a2042aa938069e862750553db0e4d82d25822c (diff) | |
Notes
Diffstat (limited to 'unittests/ExecutionEngine/Orc/RemoteObjectLayerTest.cpp')
| -rw-r--r-- | unittests/ExecutionEngine/Orc/RemoteObjectLayerTest.cpp | 34 |
1 files changed, 16 insertions, 18 deletions
diff --git a/unittests/ExecutionEngine/Orc/RemoteObjectLayerTest.cpp b/unittests/ExecutionEngine/Orc/RemoteObjectLayerTest.cpp index 819c5f8eb34b..ae408a06f84d 100644 --- a/unittests/ExecutionEngine/Orc/RemoteObjectLayerTest.cpp +++ b/unittests/ExecutionEngine/Orc/RemoteObjectLayerTest.cpp @@ -24,8 +24,7 @@ public: using ObjHandleT = uint64_t; - using ObjectPtr = - std::shared_ptr<object::OwningBinary<object::ObjectFile>>; + using ObjectPtr = std::unique_ptr<MemoryBuffer>; using LookupFn = std::function<JITSymbol(StringRef, bool)>; using SymbolLookupTable = std::map<ObjHandleT, LookupFn>; @@ -43,7 +42,7 @@ public: Expected<ObjHandleT> addObject(ObjectPtr Obj, std::shared_ptr<JITSymbolResolver> Resolver) { - return AddObject(Obj, SymTab); + return AddObject(std::move(Obj), SymTab); } Error removeObject(ObjHandleT H) { @@ -102,8 +101,7 @@ MockObjectLayer::ObjectPtr createTestObject() { B.CreateRet(ConstantInt::getSigned(Type::getInt32Ty(Ctx), 42)); SimpleCompiler IRCompiler(*TM); - return std::make_shared<object::OwningBinary<object::ObjectFile>>( - IRCompiler(*MB.getModule())); + return IRCompiler(*MB.getModule()); } TEST(RemoteObjectLayer, AddObject) { @@ -121,7 +119,7 @@ TEST(RemoteObjectLayer, AddObject) { // Copy the bytes out of the test object: the copy will be used to verify // that the original is correctly transmitted over RPC to the mock layer. - StringRef ObjBytes = TestObject->getBinary()->getData(); + StringRef ObjBytes = TestObject->getBuffer(); std::vector<char> ObjContents(ObjBytes.size()); std::copy(ObjBytes.begin(), ObjBytes.end(), ObjContents.begin()); @@ -134,7 +132,7 @@ TEST(RemoteObjectLayer, AddObject) { MockObjectLayer::SymbolLookupTable &SymTab) { // Check that the received object file content matches the original. - StringRef RPCObjContents = Obj->getBinary()->getData(); + StringRef RPCObjContents = Obj->getBuffer(); EXPECT_EQ(RPCObjContents.size(), ObjContents.size()) << "RPC'd object file has incorrect size"; EXPECT_TRUE(std::equal(RPCObjContents.begin(), RPCObjContents.end(), @@ -159,7 +157,7 @@ TEST(RemoteObjectLayer, AddObject) { }); cantFail(Client.addObject(std::move(TestObject), - std::make_shared<NullResolver>())); + std::make_shared<NullLegacyResolver>())); cantFail(ClientEP.callB<remote::utils::TerminateSession>()); ServerThread.join(); } @@ -205,8 +203,8 @@ TEST(RemoteObjectLayer, AddObjectFailure) { cantFail(ServerEP.handleOne()); }); - auto HandleOrErr = - Client.addObject(std::move(TestObject), std::make_shared<NullResolver>()); + auto HandleOrErr = Client.addObject(std::move(TestObject), + std::make_shared<NullLegacyResolver>()); EXPECT_FALSE(HandleOrErr) << "Expected error from addObject"; @@ -258,8 +256,8 @@ TEST(RemoteObjectLayer, RemoveObject) { cantFail(ServerEP.handleOne()); }); - auto H = cantFail(Client.addObject(std::move(TestObject), - std::make_shared<NullResolver>())); + auto H = cantFail(Client.addObject(std::move(TestObject), + std::make_shared<NullLegacyResolver>())); cantFail(Client.removeObject(H)); @@ -309,8 +307,8 @@ TEST(RemoteObjectLayer, RemoveObjectFailure) { cantFail(ServerEP.handleOne()); }); - auto H = cantFail(Client.addObject(std::move(TestObject), - std::make_shared<NullResolver>())); + auto H = cantFail(Client.addObject(std::move(TestObject), + std::make_shared<NullLegacyResolver>())); auto Err = Client.removeObject(H); EXPECT_TRUE(!!Err) << "Expected error from removeObject"; @@ -374,7 +372,7 @@ TEST(RemoteObjectLayer, FindSymbol) { }); cantFail(Client.addObject(std::move(TestObject), - std::make_shared<NullResolver>())); + std::make_shared<NullLegacyResolver>())); // Check that we can find and materialize a valid symbol. auto Sym1 = Client.findSymbol("foobar", true); @@ -463,7 +461,7 @@ TEST(RemoteObjectLayer, FindSymbolIn) { }); auto H = cantFail(Client.addObject(std::move(TestObject), - std::make_shared<NullResolver>())); + std::make_shared<NullLegacyResolver>())); auto Sym1 = Client.findSymbolIn(H, "foobar", true); @@ -523,7 +521,7 @@ TEST(RemoteObjectLayer, EmitAndFinalize) { }); auto H = cantFail(Client.addObject(std::move(TestObject), - std::make_shared<NullResolver>())); + std::make_shared<NullLegacyResolver>())); auto Err = Client.emitAndFinalize(H); EXPECT_FALSE(!!Err) << "emitAndFinalize should work"; @@ -573,7 +571,7 @@ TEST(RemoteObjectLayer, EmitAndFinalizeFailure) { }); auto H = cantFail(Client.addObject(std::move(TestObject), - std::make_shared<NullResolver>())); + std::make_shared<NullLegacyResolver>())); auto Err = Client.emitAndFinalize(H); EXPECT_TRUE(!!Err) << "emitAndFinalize should work"; |
