r365877 - [Driver] Delete --no-add-needed for RHEL7 or newer

Fangrui Song via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 11 23:46:47 PDT 2019


Author: maskray
Date: Thu Jul 11 23:46:47 2019
New Revision: 365877

URL: http://llvm.org/viewvc/llvm-project?rev=365877&view=rev
Log:
[Driver] Delete --no-add-needed for RHEL7 or newer

This is really not needed. gcc driver doesn't add this option.
BTW, since binutils 2.22, --no-copy-dt-needed-entries (new name of
--no-add-needed) is the default.

Modified:
    cfe/trunk/lib/Driver/ToolChains/Linux.cpp

Modified: cfe/trunk/lib/Driver/ToolChains/Linux.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains/Linux.cpp?rev=365877&r1=365876&r2=365877&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChains/Linux.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChains/Linux.cpp Thu Jul 11 23:46:47 2019
@@ -300,9 +300,6 @@ Linux::Linux(const Driver &D, const llvm
       ExtraOpts.push_back("--hash-style=both");
   }
 
-  if (Distro.IsRedhat() && Distro != Distro::RHEL5 && Distro != Distro::RHEL6)
-    ExtraOpts.push_back("--no-add-needed");
-
 #ifdef ENABLE_LINKER_BUILD_ID
   ExtraOpts.push_back("--build-id");
 #endif




More information about the cfe-commits mailing list