From: Julien Lerouge Date: Wed, 22 Oct 2008 16:30:41 +0000 (+0000) Subject: Fix for PR2881: fix a small leak exposed by valgrind, using a ManagedStatic. X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=e7d3af54737b7787c92ea03588dd4301b29d0899;p=oota-llvm.git Fix for PR2881: fix a small leak exposed by valgrind, using a ManagedStatic. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@57984 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Support/PluginLoader.cpp b/lib/Support/PluginLoader.cpp index 0635a9d52ca..5acf1d13ee9 100644 --- a/lib/Support/PluginLoader.cpp +++ b/lib/Support/PluginLoader.cpp @@ -12,6 +12,7 @@ //===----------------------------------------------------------------------===// #define DONT_GET_PLUGIN_LOADER_OPTION +#include "llvm/Support/ManagedStatic.h" #include "llvm/Support/PluginLoader.h" #include "llvm/Support/Streams.h" #include "llvm/System/DynamicLibrary.h" @@ -19,12 +20,9 @@ #include using namespace llvm; -static std::vector *Plugins; +static ManagedStatic > Plugins; void PluginLoader::operator=(const std::string &Filename) { - if (!Plugins) - Plugins = new std::vector(); - std::string Error; if (sys::DynamicLibrary::LoadLibraryPermanently(Filename.c_str(), &Error)) { cerr << "Error opening '" << Filename << "': " << Error @@ -35,10 +33,11 @@ void PluginLoader::operator=(const std::string &Filename) { } unsigned PluginLoader::getNumPlugins() { - return Plugins ? Plugins->size() : 0; + return Plugins.isConstructed() ? Plugins->size() : 0; } std::string &PluginLoader::getPlugin(unsigned num) { - assert(Plugins && num < Plugins->size() && "Asking for an out of bounds plugin"); + assert(Plugins.isConstructed() && num < Plugins->size() && + "Asking for an out of bounds plugin"); return (*Plugins)[num]; }