ro marked an inline comment as done.
ro added inline comments.
================
Comment at: lib/Driver/ToolChains/CommonArgs.cpp:528
StringRef Sanitizer) {
+ // Solaris ld doesn't need this. Inhibit use of non-existant
+ // --export-dynamic.
----------------
alekseyshl wrote:
> ro wrote:
> > alekseyshl wrote:
> > > Can you elaborate why Solaris ld does not need dynamic lists? How does it
> > > export sanitizer symbols then?
> > Solaris ld simply defaults to --export-dynamic.
> Can we say then that ld "defaults to --export-dynamic behavior" (as
> --export-dynamic does not exists there) instead of "doesn't need this"?
Sure, that's certainly clearer. As in the revised patch?
Repository:
rC Clang
https://reviews.llvm.org/D40903
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
