Fixes for a small number of compiler warnings

The ehdr_start_save variable does not need to be initialized.  However,
not initializing it will trigger a compiler warning when using older
versions of GCC.  Self-assignment unfortunately doesn't work for Clang
as Clang has a warning similar to -Winit-self as part of -Wall.

	* emultempl/elf32.em (gld*_before_allocation): Zero-initialize
	the ehdr_start_save variable.

diff --git a/ld/ChangeLog b/ld/ChangeLog
index 5d120f1dec7221f91fdb700dcaf85227000eb112..32c5f1c4316ee89372b1875e6c8cc94f77e3972d 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,8 @@
+2015-04-01  Ed Schouten  <ed@nuxi.nl>
+
+       * emultempl/elf32.em (gld*_before_allocation): Zero-initialize
+       the ehdr_start_save variable.
+
 2015-03-31  Ed Schouten  <ed@nuxi.nl>
 
        * Makefile.am (ALL_64_EMULATION_SOURCES): Add

diff --git a/ld/emultempl/elf32.em b/ld/emultempl/elf32.em
index 618e20a2519e17ccdbd2ebd330e08949455671d4..2caddbe7f165261724f6cf2128a046e84b2c21ba 100644 (file)
--- a/ld/emultempl/elf32.em
+++ b/ld/emultempl/elf32.em
@@ -1412,7 +1412,7 @@ gld${EMULATION_NAME}_before_allocation (void)
   asection *sinterp;
   bfd *abfd;
   struct elf_link_hash_entry *ehdr_start = NULL;
-  struct bfd_link_hash_entry ehdr_start_save = ehdr_start_save;
+  struct bfd_link_hash_entry ehdr_start_save = {};
 
   if (is_elf_hash_table (link_info.hash))
     {