[PATCH] D40903: [Sanitizers] Basic Solaris sanitizer support (PR 33274)
Rainer Orth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 13 01:53:14 PST 2017
ro added inline comments.
================
Comment at: lib/Driver/ToolChains/CommonArgs.cpp:684
if (AddExportDynamic)
- CmdArgs.push_back("-export-dynamic");
+ CmdArgs.push_back("--export-dynamic");
----------------
alekseyshl wrote:
> If it does not exist on Solaris, why change it then? For the consistency sake?
Consistency for one. Besides, I'm talking to the Solaris linker
engineers whether to just accept and ignore that option
for GNU ld compatibility.
Repository:
rC Clang
https://reviews.llvm.org/D40903
More information about the cfe-commits
mailing list