[PATCH] D33490: Export the required symbol from DynamicLibraryTests

Momchil Velikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 00:07:53 PDT 2017


chill added a comment.

In https://reviews.llvm.org/D33490#763597, @marsupial wrote:

> Looks fine as long as you're confident this doesn't interfere when built without LLVM_EXPORT_SYMBOLS_FOR_PLUGINS.


Yes, regardless of the state of `LLVM_EXPORT_SYMBOLS_FOR_PLUGINS`, the build uses the given symbols list instead of
exporting all the symbols or exporting whatever symbols `extract_symbols.py` decides to export.


https://reviews.llvm.org/D33490





More information about the llvm-commits mailing list