[PATCH] GCC compatibility: pass -z linker options to the linker
    Reid Kleckner 
    rnk at google.com
       
    Tue Jul 15 18:03:28 PDT 2014
    
    
  
lgtm with suggested tweaks.
================
Comment at: lib/Driver/Tools.cpp:206
@@ +205,3 @@
+      // Pass -z prefix for gcc linker compatibility.
+      A.claim(); A.render(Args, CmdArgs);
+    } else {
----------------
I was just being terse, I didn't mean to literally put them on the same line.  :)
================
Comment at: lib/Driver/Tools.cpp:7606-7609
@@ -7604,1 +7605,6 @@
+      CmdArgs.push_back(Input.getFilename()); 
+    else if (Input.getInputArg().getOption().matches(options::OPT_z)) {
+      // Pass -z prefix for gcc linker compatibility.
+      Input.getInputArg().claim(); Input.getInputArg().render(Args, CmdArgs);
+    } else {
       Input.getInputArg().renderAsInput(Args, CmdArgs);
----------------
Please revert this part of the change.  There is no reason we should forward -z to the MSVC linker.
http://reviews.llvm.org/D4393
    
    
More information about the cfe-commits
mailing list