[PATCH] D35331: [Solaris] enable --whole-archive for shared-library build, disable --version-script for Solaris-ld

Fedor Sergeev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 18 16:12:24 PDT 2017


fedor.sergeev added a reviewer: cbergstrom.
fedor.sergeev added a comment.

In https://reviews.llvm.org/D35331#813546, @ruiu wrote:

> I don't know the answer, but git blame shows me that the file was mostly by Chris Bieneman, so he might be a good person to review.


If I get it right the question everybody is asking here is not about the file and changes per se.
Rather it is about my assertions on Solaris (linker) behavior.

So I need to find Solaris expert rather than makefiles guru to get a solid review for this.


https://reviews.llvm.org/D35331





More information about the llvm-commits mailing list