gas/testsuite/

	* gas/i386/x86-64-stack.s: Add cases for push segment register.
	* gas/i386/x86-64-stack.d: Updated.
	* gas/i386/x86-64-stack-suffix.d: Updated.
	* gas/i386/x86-64-stack-intel.d: Updated.
	* gas/i386/ilp32/x86-64-stack.d: Updated.
	* gas/i386/ilp32/x86-64-stack-suffix.d: Updated.
	* gas/i386/ilp32/x86-64-stack-intel.d: Updated.

opcodes/
	* i386-dis.c (print_insn): Print spaces between multiple excess
	prefixes.  Return actual number of excess prefixes consumed,
	not always one.

	* i386-dis.c (OP_REG): Ignore REX_B for segment register cases.

diff --git a/gas/testsuite/ChangeLog b/gas/testsuite/ChangeLog
index 80c03cf..026750e 100644 (file)
--- a/gas/testsuite/ChangeLog
+++ b/gas/testsuite/ChangeLog
@@ -41,6 +41,14 @@
 
 2012-08-06  Roland McGrath  <mcgrathr@google.com>
 
+       * gas/i386/x86-64-stack.s: Add cases for push segment register.
+       * gas/i386/x86-64-stack.d: Updated.
+       * gas/i386/x86-64-stack-suffix.d: Updated.
+       * gas/i386/x86-64-stack-intel.d: Updated.
+       * gas/i386/ilp32/x86-64-stack.d: Updated.
+       * gas/i386/ilp32/x86-64-stack-suffix.d: Updated.
+       * gas/i386/ilp32/x86-64-stack-intel.d: Updated.
+
        * gas/i386/x86-64-stack.s: Add cases for push immediate.
        * gas/testsuite/gas/i386/ilp32/x86-64-stack-intel.d: Updated.
        * gas/testsuite/gas/i386/ilp32/x86-64-stack-suffix.d: Updated.

diff --git a/gas/testsuite/gas/i386/ilp32/x86-64-stack-intel.d b/gas/testsuite/gas/i386/ilp32/x86-64-stack-intel.d
index fbcada2..f98b8cd 100644 (file)
--- a/gas/testsuite/gas/i386/ilp32/x86-64-stack-intel.d
+++ b/gas/testsuite/gas/i386/ilp32/x86-64-stack-intel.d
@@ -56,5 +56,15 @@ Disassembly of section .text:
 [      ]*[a-f0-9]+:    03 04 48                add    eax,DWORD PTR \[rax\+rcx\*2\]
 [      ]*[a-f0-9]+:    68 01 02 03 04          push   0x4030201
 [      ]*[a-f0-9]+:    66 48 68 01 02 03 04    data32 rex.W push 0x4030201
+[      ]*[a-f0-9]+:    0f a8                   push   gs
+[      ]*[a-f0-9]+:    66 0f a8                pushw  gs
+[      ]*[a-f0-9]+:    48 0f a8                rex.W push gs
+[      ]*[a-f0-9]+:    66 48 0f a8             data32 rex.W push gs
+[      ]*[a-f0-9]+:    41 0f a8                rex.B push gs
+[      ]*[a-f0-9]+:    66 41 0f a8             rex.B pushw gs
+[      ]*[a-f0-9]+:    48                      rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B push gs
+[      ]*[a-f0-9]+:    66 48                   data16 rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B push gs
 [      ]*[a-f0-9]+:    90                      nop
 #pass

diff --git a/gas/testsuite/gas/i386/ilp32/x86-64-stack-suffix.d b/gas/testsuite/gas/i386/ilp32/x86-64-stack-suffix.d
index dd2f3da..cecab6d 100644 (file)
--- a/gas/testsuite/gas/i386/ilp32/x86-64-stack-suffix.d
+++ b/gas/testsuite/gas/i386/ilp32/x86-64-stack-suffix.d
@@ -56,5 +56,15 @@ Disassembly of section .text:
 [      ]*[a-f0-9]+:    03 04 48                addl   \(%rax,%rcx,2\),%eax
 [      ]*[a-f0-9]+:    68 01 02 03 04          pushq  \$0x4030201
 [      ]*[a-f0-9]+:    66 48 68 01 02 03 04    data32 rex.W pushq \$0x4030201
+[      ]*[a-f0-9]+:    0f a8                   pushq  %gs
+[      ]*[a-f0-9]+:    66 0f a8                pushw  %gs
+[      ]*[a-f0-9]+:    48 0f a8                rex.W pushq %gs
+[      ]*[a-f0-9]+:    66 48 0f a8             data32 rex.W pushq %gs
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
+[      ]*[a-f0-9]+:    66 41 0f a8             rex.B pushw %gs
+[      ]*[a-f0-9]+:    48                      rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
+[      ]*[a-f0-9]+:    66 48                   data16 rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
 [      ]*[a-f0-9]+:    90                      nop
 #pass

diff --git a/gas/testsuite/gas/i386/ilp32/x86-64-stack.d b/gas/testsuite/gas/i386/ilp32/x86-64-stack.d
index 9f4553a..fd649e2 100644 (file)
--- a/gas/testsuite/gas/i386/ilp32/x86-64-stack.d
+++ b/gas/testsuite/gas/i386/ilp32/x86-64-stack.d
@@ -56,5 +56,15 @@ Disassembly of section .text:
 [      ]*[a-f0-9]+:    03 04 48                add    \(%rax,%rcx,2\),%eax
 [      ]*[a-f0-9]+:    68 01 02 03 04          pushq  \$0x4030201
 [      ]*[a-f0-9]+:    66 48 68 01 02 03 04    data32 rex.W pushq \$0x4030201
+[      ]*[a-f0-9]+:    0f a8                   pushq  %gs
+[      ]*[a-f0-9]+:    66 0f a8                pushw  %gs
+[      ]*[a-f0-9]+:    48 0f a8                rex.W pushq %gs
+[      ]*[a-f0-9]+:    66 48 0f a8             data32 rex.W pushq %gs
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
+[      ]*[a-f0-9]+:    66 41 0f a8             rex.B pushw %gs
+[      ]*[a-f0-9]+:    48                      rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
+[      ]*[a-f0-9]+:    66 48                   data16 rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
 [      ]*[a-f0-9]+:    90                      nop
 #pass

diff --git a/gas/testsuite/gas/i386/x86-64-stack-intel.d b/gas/testsuite/gas/i386/x86-64-stack-intel.d
index cb9ee89..1902337 100644 (file)
--- a/gas/testsuite/gas/i386/x86-64-stack-intel.d
+++ b/gas/testsuite/gas/i386/x86-64-stack-intel.d
@@ -56,5 +56,15 @@ Disassembly of section .text:
 [      ]*[a-f0-9]+:    03 04 48                add    eax,DWORD PTR \[rax\+rcx\*2\]
 [      ]*[a-f0-9]+:    68 01 02 03 04          push   0x4030201
 [      ]*[a-f0-9]+:    66 48 68 01 02 03 04    data32 rex.W push 0x4030201
+[      ]*[a-f0-9]+:    0f a8                   push   gs
+[      ]*[a-f0-9]+:    66 0f a8                pushw  gs
+[      ]*[a-f0-9]+:    48 0f a8                rex.W push gs
+[      ]*[a-f0-9]+:    66 48 0f a8             data32 rex.W push gs
+[      ]*[a-f0-9]+:    41 0f a8                rex.B push gs
+[      ]*[a-f0-9]+:    66 41 0f a8             rex.B pushw gs
+[      ]*[a-f0-9]+:    48                      rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B push gs
+[      ]*[a-f0-9]+:    66 48                   data16 rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B push gs
 [      ]*[a-f0-9]+:    90                      nop
 #pass

diff --git a/gas/testsuite/gas/i386/x86-64-stack-suffix.d b/gas/testsuite/gas/i386/x86-64-stack-suffix.d
index a0b94d0..1681d79 100644 (file)
--- a/gas/testsuite/gas/i386/x86-64-stack-suffix.d
+++ b/gas/testsuite/gas/i386/x86-64-stack-suffix.d
@@ -56,5 +56,15 @@ Disassembly of section .text:
 [      ]*[a-f0-9]+:    03 04 48                addl   \(%rax,%rcx,2\),%eax
 [      ]*[a-f0-9]+:    68 01 02 03 04          pushq  \$0x4030201
 [      ]*[a-f0-9]+:    66 48 68 01 02 03 04    data32 rex.W pushq \$0x4030201
+[      ]*[a-f0-9]+:    0f a8                   pushq  %gs
+[      ]*[a-f0-9]+:    66 0f a8                pushw  %gs
+[      ]*[a-f0-9]+:    48 0f a8                rex.W pushq %gs
+[      ]*[a-f0-9]+:    66 48 0f a8             data32 rex.W pushq %gs
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
+[      ]*[a-f0-9]+:    66 41 0f a8             rex.B pushw %gs
+[      ]*[a-f0-9]+:    48                      rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
+[      ]*[a-f0-9]+:    66 48                   data16 rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
 [      ]*[a-f0-9]+:    90                      nop
 #pass

diff --git a/gas/testsuite/gas/i386/x86-64-stack.d b/gas/testsuite/gas/i386/x86-64-stack.d
index 76f7151..760d769 100644 (file)
--- a/gas/testsuite/gas/i386/x86-64-stack.d
+++ b/gas/testsuite/gas/i386/x86-64-stack.d
@@ -55,5 +55,15 @@ Disassembly of section .text:
 [      ]*[a-f0-9]+:    03 04 48                add    \(%rax,%rcx,2\),%eax
 [      ]*[a-f0-9]+:    68 01 02 03 04          pushq  \$0x4030201
 [      ]*[a-f0-9]+:    66 48 68 01 02 03 04    data32 rex.W pushq \$0x4030201
+[      ]*[a-f0-9]+:    0f a8                   pushq  %gs
+[      ]*[a-f0-9]+:    66 0f a8                pushw  %gs
+[      ]*[a-f0-9]+:    48 0f a8                rex.W pushq %gs
+[      ]*[a-f0-9]+:    66 48 0f a8             data32 rex.W pushq %gs
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
+[      ]*[a-f0-9]+:    66 41 0f a8             rex.B pushw %gs
+[      ]*[a-f0-9]+:    48                      rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
+[      ]*[a-f0-9]+:    66 48                   data16 rex.W
+[      ]*[a-f0-9]+:    41 0f a8                rex.B pushq %gs
 [      ]*[a-f0-9]+:    90                      nop
 #pass

diff --git a/gas/testsuite/gas/i386/x86-64-stack.s b/gas/testsuite/gas/i386/x86-64-stack.s
index 2da658b..0b8707e 100644 (file)
--- a/gas/testsuite/gas/i386/x86-64-stack.s
+++ b/gas/testsuite/gas/i386/x86-64-stack.s
@@ -29,6 +29,11 @@ _start:
        # push with a 4-byte immediate
        try     0x68, 0x01, 0x02, 0x03, 0x04
 
+       # push a segment register
+       try     0x0f, 0xa8
+       # with extraneous rex.B
+       try     0x41, 0x0f, 0xa8
+
        # This is just to synchronize the disassembly.
        # Any new cases must come before this line!
        nop

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index c90599b..25d53fc 100644 (file)
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,4 +1,12 @@
 2012-08-06  Roland McGrath  <mcgrathr@google.com>
+
+       * i386-dis.c (print_insn): Print spaces between multiple excess
+       prefixes.  Return actual number of excess prefixes consumed,
+       not always one.
+
+       * i386-dis.c (OP_REG): Ignore REX_B for segment register cases.
+
+2012-08-06  Roland McGrath  <mcgrathr@google.com>
            Victor Khimenko  <khim@google.com>
            H.J. Lu  <hongjiu.lu@intel.com>
 

diff --git a/opcodes/i386-dis.c b/opcodes/i386-dis.c
index 43d7ac3..da5ede5 100644 (file)
--- a/opcodes/i386-dis.c
+++ b/opcodes/i386-dis.c
@@ -11450,9 +11450,10 @@ print_insn (bfd_vma pc, disassemble_info *info)
       for (i = 0;
           i < (int) ARRAY_SIZE (all_prefixes) && all_prefixes[i];
           i++)
-       (*info->fprintf_func) (info->stream, "%s",
+       (*info->fprintf_func) (info->stream, "%s%s",
+                               i == 0 ? "" : " ",
                               prefix_name (all_prefixes[i], sizeflag));
-      return 1;
+      return i;
     }
 
   insn_codep = codep;
@@ -13471,6 +13472,15 @@ OP_REG (int code, int sizeflag)
 {
   const char *s;
   int add;
+
+  switch (code)
+    {
+    case es_reg: case ss_reg: case cs_reg:
+    case ds_reg: case fs_reg: case gs_reg:
+      oappend (names_seg[code - es_reg]);
+      return;
+    }
+
   USED_REX (REX_B);
   if (rex & REX_B)
     add = 8;
@@ -13483,10 +13493,6 @@ OP_REG (int code, int sizeflag)
     case sp_reg: case bp_reg: case si_reg: case di_reg:
       s = names16[code - ax_reg + add];
       break;
-    case es_reg: case ss_reg: case cs_reg:
-    case ds_reg: case fs_reg: case gs_reg:
-      s = names_seg[code - es_reg + add];
-      break;
     case al_reg: case ah_reg: case cl_reg: case ch_reg:
     case dl_reg: case dh_reg: case bl_reg: case bh_reg:
       USED_REX (0);