[llvm] 9e66d54 - [ORC] Remove COFFPlatform::DylibsToPreload. NFC.
Lang Hames via llvm-commits
llvm-commits at lists.llvm.org
Mon May 26 20:46:18 PDT 2025
Author: Lang Hames
Date: 2025-05-27T13:46:12+10:00
New Revision: 9e66d54ed4bbbaad0175aa0758a18b31e16334a3
URL: https://github.com/llvm/llvm-project/commit/9e66d54ed4bbbaad0175aa0758a18b31e16334a3
DIFF: https://github.com/llvm/llvm-project/commit/9e66d54ed4bbbaad0175aa0758a18b31e16334a3.diff
LOG: [ORC] Remove COFFPlatform::DylibsToPreload. NFC.
DylibsToPreload is only used in the constructor. This patch makes it a local
variable.
Added:
Modified:
llvm/include/llvm/ExecutionEngine/Orc/COFFPlatform.h
llvm/lib/ExecutionEngine/Orc/COFFPlatform.cpp
Removed:
################################################################################
diff --git a/llvm/include/llvm/ExecutionEngine/Orc/COFFPlatform.h b/llvm/include/llvm/ExecutionEngine/Orc/COFFPlatform.h
index cebab0f2b30a3..acfefbcddc49e 100644
--- a/llvm/include/llvm/ExecutionEngine/Orc/COFFPlatform.h
+++ b/llvm/include/llvm/ExecutionEngine/Orc/COFFPlatform.h
@@ -202,8 +202,6 @@ class COFFPlatform : public Platform {
DenseMap<JITDylib *, SymbolLookupSet> RegisteredInitSymbols;
- std::set<std::string> DylibsToPreload;
-
std::mutex PlatformMutex;
};
diff --git a/llvm/lib/ExecutionEngine/Orc/COFFPlatform.cpp b/llvm/lib/ExecutionEngine/Orc/COFFPlatform.cpp
index 9ff12b201a988..ce04062598b52 100644
--- a/llvm/lib/ExecutionEngine/Orc/COFFPlatform.cpp
+++ b/llvm/lib/ExecutionEngine/Orc/COFFPlatform.cpp
@@ -401,6 +401,7 @@ COFFPlatform::COFFPlatform(
}
VCRuntimeBootstrap = std::move(*VCRT);
+ std::set<std::string> DylibsToPreload;
for (auto &Lib : OrcRuntimeGenerator->getImportedDynamicLibraries())
DylibsToPreload.insert(Lib);
More information about the llvm-commits
mailing list