[PATCH] Make the plugin loader limit its symbol search to the specified module.

Reid Kleckner rnk at google.com
Tue Aug 26 15:31:13 PDT 2014

Comment at: include/llvm/Support/DynamicLibrary.h:48
@@ -45,2 +47,3 @@
-    explicit DynamicLibrary(void *data = &Invalid) : Data(data) {}
+    llvm::SmallString<128> Filename;
It looks like DynamicLibrary is supposed to be a thin value type wrapper around native library handles. Adding this string makes it no longer trivially copyable and blows out its size. If we really need this, I think we should delete the copy ctor and eliminate the places where we pass it around by value.


More information about the llvm-commits mailing list