[llvm-commits] CVS: llvm/lib/Support/PluginLoader.cpp

[Chris Lattner]

Changes in directory llvm/lib/Support:

PluginLoader.cpp updated: 1.10 -> 1.11

---
Log message:

Implicitly getting a new option by linking to support.o instead of support.a
is a bad idea.  Make tools that want the option #include PluginSupport.h 
explicitly.


---
Diffs of the changes:  (+8 -23)

Index: llvm/lib/Support/PluginLoader.cpp
diff -u llvm/lib/Support/PluginLoader.cpp:1.10 llvm/lib/Support/PluginLoader.cpp:1.11
--- llvm/lib/Support/PluginLoader.cpp:1.10	Fri May 28 18:35:39 2004
+++ llvm/lib/Support/PluginLoader.cpp	Sat Jul 10 20:04:33 2004
@@ -7,34 +7,19 @@
 // 
 //===----------------------------------------------------------------------===//
 //
-// This file implements the -load <plugin> command line option processor.  When
-// linked into a program, this new command line option is available that allows
-// users to load shared objects into the running program.
-//
-// Note that there are no symbols exported by the .o file generated for this
-// .cpp file.  Because of this, a program must link against support.o instead of
-// support.a: otherwise this translation unit will not be included.
+// This file implements the -load <plugin> command line option handler.
 //
 //===----------------------------------------------------------------------===//
 
+#define DONT_GET_PLUGIN_LOADER_OPTION
+#include "Support/PluginLoader.h"
 #include "Support/DynamicLinker.h"
-#include "Support/CommandLine.h"
-#include "Config/config.h"
 #include <iostream>
 using namespace llvm;
 
-namespace {
-  struct PluginLoader {
-    void operator=(const std::string &Filename) {
-      std::string ErrorMessage;
-      if (LinkDynamicObject (Filename.c_str (), &ErrorMessage))
-        std::cerr << "Error opening '" << Filename << "': " << ErrorMessage
-                  << "\n  -load request ignored.\n";	
-    }
-  };
+void PluginLoader::operator=(const std::string &Filename) {
+  std::string ErrorMessage;
+  if (LinkDynamicObject(Filename.c_str(), &ErrorMessage))
+    std::cerr << "Error opening '" << Filename << "': " << ErrorMessage
+              << "\n  -load request ignored.\n";	
 }
-
-// This causes operator= above to be invoked for every -load option.
-static cl::opt<PluginLoader, false, cl::parser<std::string> >
-LoadOpt("load", cl::ZeroOrMore, cl::value_desc("plugin" SHLIBEXT),
-        cl::desc("Load the specified plugin"));