gold: enable new dtags by default

The "new" dtags options have been around for 14+ years, and for all the
targets that gold supports, these flags have always existed.  So enable
them by default.

Having behavior be different from ld.bfd isn't new, and this behavior
is the "better" one, so there shouldn't be a problem based on that.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 42d2d03..8e70fdd 100644 (file)
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,5 +1,9 @@
 2013-01-18  Mike Frysinger  <vapier@gentoo.org>
 
+       * options.h (General_options): Change default to true for new_dtags.
+
+2013-01-18  Mike Frysinger  <vapier@gentoo.org>
+
        * layout.cc (Layout::finish_dynamic_section): Only add DT_RPATH
        when enable_new_dtags is false.  Only add DT_RUNPATH when
        enable_new_dtags is true.

diff --git a/gold/options.h b/gold/options.h
index 1eb2da2..9e65e8d 100644 (file)
--- a/gold/options.h
+++ b/gold/options.h
@@ -904,7 +904,7 @@ class General_options
              N_("Do not page align data, do not make text readonly"),
              N_("Page align data, make text readonly"));
 
-  DEFINE_enable(new_dtags, options::EXACTLY_TWO_DASHES, '\0', false,
+  DEFINE_enable(new_dtags, options::EXACTLY_TWO_DASHES, '\0', true,
                N_("Enable use of DT_RUNPATH and DT_FLAGS"),
                N_("Disable use of DT_RUNPATH and DT_FLAGS"));