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 <[email protected]>

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 42d2d03af0a02d45d99c8be705f8f4f9bd75bd31..8e70fddc6fff48e5e9e0780f6949ffb2127a52c3 100644 (file)
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,7 @@
+2013-01-18  Mike Frysinger  <[email protected]>
+
+       * options.h (General_options): Change default to true for new_dtags.
+
 2013-01-18  Mike Frysinger  <[email protected]>
 
        * layout.cc (Layout::finish_dynamic_section): Only add DT_RPATH

diff --git a/gold/options.h b/gold/options.h
index 1eb2da207d5db970254370d1691b54d7faf0d4dc..9e65e8d878dbdfa339856e08044594d1b0b56b8c 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"));