gas/testsuite/

2005-01-12  H.J. Lu  <hongjiu.lu@intel.com>

	* i386/i386.exp: Run "sib".

	* gas/i386/sib.d: New file.
	* gas/i386/sib.s: Likewise.

opcodes/

2005-01-12  H.J. Lu  <hongjiu.lu@intel.com>

	* i386-dis.c (OP_E): Ignore scale when index == 0x4 in SIB.

diff --git a/gas/testsuite/ChangeLog b/gas/testsuite/ChangeLog
index 24c5cc85e6bb7ff9ef4717ec1283302a57466538..ed357fb312024374e1efa486214feac71baa74b7 100644 (file)
--- a/gas/testsuite/ChangeLog
+++ b/gas/testsuite/ChangeLog
@@ -1,3 +1,10 @@
+2005-01-12  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * i386/i386.exp: Run "sib".
+
+       * gas/i386/sib.d: New file.
+       * gas/i386/sib.s: Likewise.
+
 2005-01-09  Andreas Schwab  <schwab@suse.de>
 
        * gas/i386/intel16.d: Ignore trailing text with #pass. 

diff --git a/gas/testsuite/gas/i386/i386.exp b/gas/testsuite/gas/i386/i386.exp
index 184626857d437b07ec80b7517f0d5954f18a8739..98b6ea66bbca363a938ea2ffd049c5939e0be834 100644 (file)
--- a/gas/testsuite/gas/i386/i386.exp
+++ b/gas/testsuite/gas/i386/i386.exp
@@ -57,6 +57,7 @@ if [expr ([istarget "i*86-*-*"] ||  [istarget "x86_64-*-*"]) && [gas_32_check]]
     run_dump_test "sse2"
     run_dump_test "sub"
     run_dump_test "prescott"
+    run_dump_test "sib"
 
     if {![istarget "*-*-aix*"]
        && (![is_elf_format] || [istarget "*-*-linux*"]

diff --git a/gas/testsuite/gas/i386/sib.d b/gas/testsuite/gas/i386/sib.d
new file mode 100644 (file)
index 0000000..2a5dbb6
--- /dev/null
+++ b/gas/testsuite/gas/i386/sib.d
@@ -0,0 +1,15 @@
+#objdump: -dw
+#name: i386 SIB
+
+.*: +file format .*
+
+Disassembly of section .text:
+
+0+000 <foo>:
+   0:  8b 04 23 [      ]*mov [         ]*\(%ebx\),%eax
+   3:  8b 04 63 [      ]*mov [         ]*\(%ebx\),%eax
+   6:  8b 04 a3 [      ]*mov [         ]*\(%ebx\),%eax
+   9:  8b 04 e3 [      ]*mov [         ]*\(%ebx\),%eax
+   c:  90 [    ]*nop [         ]*
+   d:  90 [    ]*nop [         ]*
+       ...

diff --git a/gas/testsuite/gas/i386/sib.s b/gas/testsuite/gas/i386/sib.s
new file mode 100644 (file)
index 0000000..25d88b7
--- /dev/null
+++ b/gas/testsuite/gas/i386/sib.s
@@ -0,0 +1,11 @@
+#Test the special case of the index bits, 0x4, in SIB.
+
+       .text
+foo:
+       .byte   0x8B, 0x04, 0x23        # effect is: movl (%ebx), %eax
+       .byte   0x8B, 0x04, 0x63        # effect is: movl (%ebx), %eax  
+       .byte   0x8B, 0x04, 0xA3        # effect is: movl (%ebx), %eax
+       .byte   0x8B, 0x04, 0xE3        # effect is: movl (%ebx), %eax
+       nop
+       nop
+       .p2align        4,0

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 2effaf30654e70f1e0ddb61ff05cd6927103d4f7..1b9fc940e04ba45050273b6139ac164a5aab8580 100644 (file)
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,7 @@
+2005-01-12  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * i386-dis.c (OP_E): Ignore scale when index == 0x4 in SIB.
+
 2005-01-10  Andreas Schwab  <schwab@suse.de>
 
        * disassemble.c (disassemble_init_for_target) <case

diff --git a/opcodes/i386-dis.c b/opcodes/i386-dis.c
index b5cc638f096354c66632bd002ea3f472252abeea..c24c6561e1af1d831d2871c93b2494369348234e 100644 (file)
--- a/opcodes/i386-dis.c
+++ b/opcodes/i386-dis.c
@@ -3191,8 +3191,10 @@ OP_E (int bytemode, int sizeflag)
        {
          havesib = 1;
          FETCH_DATA (the_info, codep + 1);
-         scale = (*codep >> 6) & 3;
          index = (*codep >> 3) & 7;
+         if (index != 0x4)
+           /* When INDEX == 0x4, scale is ignored.  */
+           scale = (*codep >> 6) & 3;
          base = *codep & 7;
          USED_REX (REX_EXTY);
          USED_REX (REX_EXTZ);