* 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 1ab6a988ef2cdb264f707d0a9ed3e972ea59bbb6..9724023ee4fa9aead68f35bbc667f74097b9e33b 100644 (file)
--- 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 6601447e6933f235082d4acf5029c63c0096df57..cd7dd5f18d890a72cf5cb113106467575fe9030a 100644 (file)
--- 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