ManagedStatic: remove from PluginLoader
authorNicolai Hähnle <nicolai.haehnle@amd.com>
Wed, 29 Jun 2022 14:14:55 +0000 (16:14 +0200)
committerNicolai Hähnle <nicolai.haehnle@amd.com>
Wed, 3 Aug 2022 08:41:58 +0000 (10:41 +0200)
Differential Revision: https://reviews.llvm.org/D129123

llvm/lib/Support/PluginLoader.cpp

index 6fe195f..cbe413c 100644 (file)
 #define DONT_GET_PLUGIN_LOADER_OPTION
 #include "llvm/Support/PluginLoader.h"
 #include "llvm/Support/DynamicLibrary.h"
-#include "llvm/Support/ManagedStatic.h"
 #include "llvm/Support/Mutex.h"
 #include "llvm/Support/raw_ostream.h"
 #include <vector>
 using namespace llvm;
 
-static ManagedStatic<std::vector<std::string> > Plugins;
-static ManagedStatic<sys::SmartMutex<true> > PluginsLock;
+namespace {
+
+struct Plugins {
+  sys::SmartMutex<true> Lock;
+  std::vector<std::string> List;
+};
+
+Plugins &getPlugins() {
+  static Plugins P;
+  return P;
+}
+
+} // anonymous namespace
 
 void PluginLoader::operator=(const std::string &Filename) {
-  sys::SmartScopedLock<true> Lock(*PluginsLock);
+  auto &P = getPlugins();
+  sys::SmartScopedLock<true> Lock(P.Lock);
   std::string Error;
   if (sys::DynamicLibrary::LoadLibraryPermanently(Filename.c_str(), &Error)) {
     errs() << "Error opening '" << Filename << "': " << Error
            << "\n  -load request ignored.\n";
   } else {
-    Plugins->push_back(Filename);
+    P.List.push_back(Filename);
   }
 }
 
 unsigned PluginLoader::getNumPlugins() {
-  sys::SmartScopedLock<true> Lock(*PluginsLock);
-  return Plugins.isConstructed() ? Plugins->size() : 0;
+  auto &P = getPlugins();
+  sys::SmartScopedLock<true> Lock(P.Lock);
+  return P.List.size();
 }
 
 std::string &PluginLoader::getPlugin(unsigned num) {
-  sys::SmartScopedLock<true> Lock(*PluginsLock);
-  assert(Plugins.isConstructed() && num < Plugins->size() &&
-         "Asking for an out of bounds plugin");
-  return (*Plugins)[num];
+  auto &P = getPlugins();
+  sys::SmartScopedLock<true> Lock(P.Lock);
+  assert(num < P.List.size() && "Asking for an out of bounds plugin");
+  return P.List[num];
 }