From: Jeff Law <law@redhat.com>
Date: Fri, 24 Jul 1998 17:49:08 +0000 (+0000)
Subject:         * elf-m10300.c (mn10300_elf_relax_section): Do not relax "dmul",
X-Git-Tag: gdb-4_18~1468
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=88ac1a5e137356d81e07149f23f3e4f7eaa76c7f;p=platform%2Fupstream%2Fbinutils.git

    * elf-m10300.c (mn10300_elf_relax_section): Do not relax "dmul",
        "dmulu", "dmach", "dmachu" with 32bit operands.
---

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 1ab6a98..9724023 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,5 +1,10 @@
 Fri Jul 24 11:24:29 1998  Jeffrey A Law  (law@cygnus.com)
 
+start-sanitize-am33
+	* elf-m10300.c (mn10300_elf_relax_section): Do not relax "dmul",
+	"dmulu", "dmach", "dmachu" with 32bit operands.
+end-sanitize-am33
+
 	* elf-m10300.c (mn10300_elf_howto): Add R_MN10300_24 entry.
 	(mn10300_elf_reloc_map): Similarly.
 	(mn10300_elf_final_link_relocate): Handle R_MN10300_24.
diff --git a/bfd/elf-m10300.c b/bfd/elf-m10300.c
index 6601447..cd7dd5f1 100644
--- a/bfd/elf-m10300.c
+++ b/bfd/elf-m10300.c
@@ -1897,7 +1897,7 @@ mn10300_elf_relax_section (abfd, sec, link_info, again)
 	  bfd_vma value = symval;
 	  value += irel->r_addend;
 
-	  /* See if the value will fit in 8 bits.
+	  /* See if the value will fit in 8 bits.  */
 	  if ((long)value < 0x7f && (long)value > -0x80)
 	    {
 	      unsigned char code;
@@ -1913,9 +1913,13 @@ mn10300_elf_relax_section (abfd, sec, link_info, again)
 	          /* Get the second opcode.  */
 	          code = bfd_get_8 (abfd, contents + irel->r_offset - 2);
 
-	          if ((code & 0x0f) == 0x09 || (code & 0x0f) == 0x08
-		      || (code & 0x0f) == 0x0a || (code & 0x0f) == 0x0b
-		      || (code & 0x0f) == 0x0e)
+		  /* We can not relax 0x6b, 0x7b, 0x8b, 0x9b as no 24bit
+		     equivalent instructions exists.  */
+	          if (code != 0x6b && code != 0x7b
+		      && code != 0x8b && code != 0x9b
+		      && ((code & 0x0f) == 0x09 || (code & 0x0f) == 0x08
+			  || (code & 0x0f) == 0x0a || (code & 0x0f) == 0x0b
+			  || (code & 0x0f) == 0x0e))
 		    {
 		      /* Not safe if the high bit is on as relaxing may
 		         move the value out of high mem and thus not fit