[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