[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