[PATCH] D33529: Allow Unix libraries loaded with RTLD_LOCAL to be searched.

Vassil Vassilev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 04:08:03 PDT 2017


v.g.vassilev added inline comments.


================
Comment at: lib/Support/DynamicLibrary.cpp:85
+#if defined(LLVM_ON_WIN32) && !defined(NDEBUG)
       for (void *Handle : Handles)
         assert(!DLSym(Handle, Symbol) && "Symbol exists in non process handle");
----------------
I am a little confused by this section here. This seems well optimized for binaries which are statically linked.

IIUC we disallow searching symbols from the dlopened from the process libraries? Why?


https://reviews.llvm.org/D33529





More information about the llvm-commits mailing list