Stop strip from merging notes when stripping debug or dwo information.

	* objcopy.c (strip_main): Do not enable note merging by default if
	just stripping debug or dwo information.
	* doc/binutils.texi (strip): Update documentation.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 9aa6bf9fcabf72ff0e9365eb4a327efe374e0034..8fbc761b6a60c800f9abaf5607cdedc838d11b15 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,9 @@
+2019-04-24  Nick Clifton  <nickc@redhat.com>
+
+       * objcopy.c (strip_main): Do not enable note merging by default if
+       just stripping debug or dwo information.
+       * doc/binutils.texi (strip): Update documentation.
+
 2019-04-23  Nick Clifton  <nickc@redhat.com>
 
        * README-how-to-make-a-release: Add note to update the symbolic

diff --git a/binutils/doc/binutils.texi b/binutils/doc/binutils.texi
index 077cf0650dc4fe3632bd0d26b26349ab848e370c..502f68d5dd64f0a1024706a943bb2e7b7d1aa380 100644 (file)
--- a/binutils/doc/binutils.texi
+++ b/binutils/doc/binutils.texi
@@ -3313,7 +3313,7 @@ normally be stripped.  This option may be given more than once.
 @itemx --no-merge-notes
 For ELF files, attempt (or do not attempt) to reduce the size of any
 SHT_NOTE type sections by removing duplicate notes.  The default is to
-attempt this reduction.
+attempt this reduction unless stripping debug or DWO information.
 
 @item -N @var{symbolname}
 @itemx --strip-symbol=@var{symbolname}

diff --git a/binutils/objcopy.c b/binutils/objcopy.c
index 330b93c1ad3f87a0f5777ac88dc4af1398959bbe..673e1f640ae9800748c46456d6437bbe2ad3a648 100644 (file)
--- a/binutils/objcopy.c
+++ b/binutils/objcopy.c
@@ -4410,8 +4410,7 @@ strip_main (int argc, char *argv[])
   int c;
   int i;
   char *output_file = NULL;
-
-  merge_notes = TRUE;
+  bfd_boolean merge_notes_set = FALSE;
 
   while ((c = getopt_long (argc, argv, "I:O:F:K:MN:R:o:sSpdgxXHhVvwDU",
                           strip_options, (int *) 0)) != EOF)
@@ -4452,9 +4451,11 @@ strip_main (int argc, char *argv[])
          break;
        case 'M':
          merge_notes = TRUE;
+         merge_notes_set = TRUE;
          break;
        case OPTION_NO_MERGE_NOTES:
          merge_notes = FALSE;
+         merge_notes_set = TRUE;
          break;
        case 'N':
          add_specific_symbol (optarg, strip_specific_htab);
@@ -4506,6 +4507,16 @@ strip_main (int argc, char *argv[])
        }
     }
 
+  /* If the user has not expressly chosen to merge/not-merge ELF notes
+     then enable the merging unless we are stripping debug or dwo info.  */
+  if (! merge_notes_set
+      && (strip_symbols == STRIP_UNDEF
+         || strip_symbols == STRIP_ALL
+         || strip_symbols == STRIP_UNNEEDED
+         || strip_symbols == STRIP_NONDEBUG
+         || strip_symbols == STRIP_NONDWO))
+    merge_notes = TRUE;
+
   if (formats_info)
     {
       display_info ();