diff options
author | Lang Hames <lhames@gmail.com> | 2018-07-20 18:31:52 +0000 |
---|---|---|
committer | Lang Hames <lhames@gmail.com> | 2018-07-20 18:31:52 +0000 |
commit | d4df0f17335c125068fed51f554a5d81caebeb1a (patch) | |
tree | c9976f0c403de2d1a46735af1c6dc20f0c0588e0 /llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp | |
parent | fd0c1e71694ef1def33ec6293dbfae1f4c96f39b (diff) | |
download | bcm5719-llvm-d4df0f17335c125068fed51f554a5d81caebeb1a.tar.gz bcm5719-llvm-d4df0f17335c125068fed51f554a5d81caebeb1a.zip |
[ORC] Simplify VSO::lookupFlags to return the flags map.
This discards the unresolved symbols set and returns the flags map directly
(rather than mutating it via the first argument).
The unresolved symbols result made it easy to chain lookupFlags calls, but such
chaining should be rare to non-existant (especially now that symbol resolvers
are being deprecated) so the simpler method signature is preferable.
llvm-svn: 337594
Diffstat (limited to 'llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp')
-rw-r--r-- | llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp b/llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp index 9d4f9463372..c0afbc6be06 100644 --- a/llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp +++ b/llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp @@ -215,11 +215,8 @@ TEST_F(CoreAPIsStandardTest, LookupFlagsTest) { SymbolNameSet Names({Foo, Bar, Baz}); - SymbolFlagsMap SymbolFlags; - auto SymbolsNotFound = V.lookupFlags(SymbolFlags, Names); + auto SymbolFlags = V.lookupFlags(Names); - EXPECT_EQ(SymbolsNotFound.size(), 1U) << "Expected one not-found symbol"; - EXPECT_EQ(SymbolsNotFound.count(Baz), 1U) << "Expected Baz to be not-found"; EXPECT_EQ(SymbolFlags.size(), 2U) << "Returned symbol flags contains unexpected results"; EXPECT_EQ(SymbolFlags.count(Foo), 1U) << "Missing lookupFlags result for Foo"; |