[PATCH][x86] Fix for objdump internal error

H.J. Lu hjl.tools@gmail.com
Tue Dec 17 16:47:00 GMT 2013


On Tue, Dec 17, 2013 at 8:30 AM, Michael Zolotukhin
<michael.v.zolotukhin@gmail.com> wrote:
> Hi HJ,
>
> Attached is patch with the tests. Ok for trunk?
>
> gas/testsuite/Changelog:
> 2013-12-17  Michael Zolotukhin  <michael.v.zolotukhin@gmail.com>
>
>         * gas/i386/disassem.s: New.
>         * gas/i386/disassem.d: New.
>         * gas/i386/x86-64-disassem.s: New.
>         * gas/i386/x86-64-disassem.d: New.
>         * gas/i386/i386.exp: Call new tests.
>
>
> opcodes/Changelog:
> 2013-12-17  Michael Zolotukhin  <michael.v.zolotukhin@gmail.com>
>
>         * i386-dis.c (MOD_FF_REG_3): New.
>         (MOD_FF_REG_5): New.
>         (mod_table): Add new entries.
>         (reg_table): Fix decoding of ljmp and lcall.
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Just say use MOD_FF_REG_3 and MOD_FF_REG_5.

--- /dev/null
+++ b/gas/testsuite/gas/i386/disassem.d
@@ -0,0 +1,14 @@
+#objdump: -drw
+#name: ljmp and lcall with invalid modrm byte
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Change it to opcodes with invalid modrm byte.

+++ b/gas/testsuite/gas/i386/x86-64-disassem.d
@@ -0,0 +1,14 @@
+#objdump: -drw
+#name: ljmp and lcall with invalid modrm byte
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Change it to x86-64 opcodes with invalid modrm byte.

+++ b/gas/testsuite/gas/i386/disassem.s
@@ -0,0 +1,2 @@

Add ".text".

+.byte 0xFF, 0xEF
+.byte 0xFF, 0xD8

+++ b/gas/testsuite/gas/i386/x86-64-disassem.s

Add ".text".

@@ -0,0 +1,3 @@
+.byte 0xFF, 0xEF
+.byte 0xFF, 0xD8
+
^^^ Remove trailing blank lines.

Thanks.

-- 
H.J.



More information about the Binutils mailing list