2006-09-08 Vladimir Prus <vladimir@codesourcery.com>

	* elf32-arm.c (elf32_arm_swap_symbol_out): Remove
	unconditional setting of low bit for Thumb symbol
	mistakenly left behind after check for external
	symbols was added.  Fix comment typo.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 88cff5c8b83757e88a689c74d34294c00b5b9353..fd79708af3d0f3d5aa8def6d0ff4f38664eff11d 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,10 @@
+2006-09-08  Vladimir Prus  <vladimir@codesourcery.com>
+
+       * elf32-arm.c (elf32_arm_swap_symbol_out): Remove
+       unconditional setting of low bit for Thumb symbol
+       mistakenly left behind after check for external
+       symbols was added.  Fix comment typo.
+
 2006-09-07  H.J. Lu  <hongjiu.lu@intel.com>
 
        * elf-bfd.h (elf_link_hash_entry): Add a dynamic field.

diff --git a/bfd/elf32-arm.c b/bfd/elf32-arm.c
index 530ef9987344dc5fc4da7cd29bed007812ae10b6..8db3c8cb2fdccc97a8c1ac3a68f49798322dcfb6 100644 (file)
--- a/bfd/elf32-arm.c
+++ b/bfd/elf32-arm.c
@@ -9385,14 +9385,12 @@ elf32_arm_swap_symbol_out (bfd *abfd,
              linker will simulate the work of dynamic linker of resolving
              symbols and will carry over the thumbness of found symbols to
              the output symbol table. It's not clear how it happens, but
-             the thumbness of underfined symbols can well be different at
+             the thumbness of undefined symbols can well be different at
              runtime, and writing '1' for them will be confusing for users
              and possibly for dynamic linker itself.
           */
           newsym.st_value |= 1;
         }
-
-      newsym.st_value |= 1;
       
       src = &newsym;
     }