[llvm] r343442 - [ORC] Pass Symbols to ExecutionSession::lookup by value, potentially saving a
Lang Hames via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 30 21:59:11 PDT 2018
Author: lhames
Date: Sun Sep 30 21:59:10 2018
New Revision: 343442
URL: http://llvm.org/viewvc/llvm-project?rev=343442&view=rev
Log:
[ORC] Pass Symbols to ExecutionSession::lookup by value, potentially saving a
copy.
Modified:
llvm/trunk/include/llvm/ExecutionEngine/Orc/Core.h
llvm/trunk/lib/ExecutionEngine/Orc/Core.cpp
Modified: llvm/trunk/include/llvm/ExecutionEngine/Orc/Core.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ExecutionEngine/Orc/Core.h?rev=343442&r1=343441&r2=343442&view=diff
==============================================================================
--- llvm/trunk/include/llvm/ExecutionEngine/Orc/Core.h (original)
+++ llvm/trunk/include/llvm/ExecutionEngine/Orc/Core.h Sun Sep 30 21:59:10 2018
@@ -740,7 +740,7 @@ public:
/// dependenant symbols for this query (e.g. it is being made by a top level
/// client to get an address to call) then the value NoDependenciesToRegister
/// can be used.
- void lookup(const JITDylibList &JDs, const SymbolNameSet &Symbols,
+ void lookup(const JITDylibList &JDs, SymbolNameSet Symbols,
SymbolsResolvedCallback OnResolve, SymbolsReadyCallback OnReady,
RegisterDependenciesFunction RegisterDependencies);
Modified: llvm/trunk/lib/ExecutionEngine/Orc/Core.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/Orc/Core.cpp?rev=343442&r1=343441&r2=343442&view=diff
==============================================================================
--- llvm/trunk/lib/ExecutionEngine/Orc/Core.cpp (original)
+++ llvm/trunk/lib/ExecutionEngine/Orc/Core.cpp Sun Sep 30 21:59:10 2018
@@ -1628,7 +1628,7 @@ Expected<SymbolMap> ExecutionSession::le
}
void ExecutionSession::lookup(
- const JITDylibList &JDs, const SymbolNameSet &Symbols,
+ const JITDylibList &JDs, SymbolNameSet Symbols,
SymbolsResolvedCallback OnResolve, SymbolsReadyCallback OnReady,
RegisterDependenciesFunction RegisterDependencies) {
@@ -1640,7 +1640,7 @@ void ExecutionSession::lookup(
auto Unresolved = std::move(Symbols);
std::map<JITDylib *, MaterializationUnitList> MUsMap;
auto Q = std::make_shared<AsynchronousSymbolQuery>(
- Symbols, std::move(OnResolve), std::move(OnReady));
+ Unresolved, std::move(OnResolve), std::move(OnReady));
bool QueryIsFullyResolved = false;
bool QueryIsFullyReady = false;
bool QueryFailed = false;
More information about the llvm-commits
mailing list