There's no need for two separate opcode entries for JECXZ. And rather than having an ad-hoc test of the instruction at the end of x86-64-opcode.s, fill out the section previously established for it (but left empty). Once at it, also fill out the similar LOOP section, and correct comments in the Jcc one. gas/testsuite/ 2013-04-08 Jan Beulich * gas/i386/x86-64-opcode.s: Flesh out LOOP and J*CXZ sections. Correct comments in Jcc section. * gas/i386/x86-64-opcode.d: Refresh. * gas/i386/ilp32/x86-64-opcode.d: Refresh. opcodes/ 2013-04-08 Jan Beulich * i386-opc.tbl: Fold 64-bit and non-64-bit jecxz entries. * i386-tbl.h: Re-generate. --- 2013-04-08/gas/testsuite/gas/i386/ilp32/x86-64-opcode.d +++ 2013-04-08/gas/testsuite/gas/i386/ilp32/x86-64-opcode.d @@ -52,6 +52,11 @@ Disassembly of section .text: [ ]*[a-f0-9]+: 48 0f c3 00 movnti %rax,\(%rax\) [ ]*[a-f0-9]+: 4d 0f c3 00 movnti %r8,\(%r8\) [ ]*[a-f0-9]+: 4c 0f c3 00 movnti %r8,\(%rax\) +[ ]*[a-f0-9]+: e2 fe loop 0x[0-9a-f]+ +[ ]*[a-f0-9]+: e2 fe loop 0x[0-9a-f]+ +[ ]*[a-f0-9]+: 67 e2 fd loopl 0x[0-9a-f]+ +[ ]*[a-f0-9]+: e3 fe jrcxz 0x[0-9a-f]+ +[ ]*[a-f0-9]+: 67 e3 fd jecxz 0x[0-9a-f]+ [ ]*[a-f0-9]+: 41 f6 38 idivb \(%r8\) [ ]*[a-f0-9]+: f6 38 idivb \(%rax\) [ ]*[a-f0-9]+: 66 41 f7 38 idivw \(%r8\) --- 2013-04-08/gas/testsuite/gas/i386/x86-64-opcode.d +++ 2013-04-08/gas/testsuite/gas/i386/x86-64-opcode.d @@ -51,6 +51,11 @@ Disassembly of section .text: [ ]*[a-f0-9]+: 48 0f c3 00 movnti %rax,\(%rax\) [ ]*[a-f0-9]+: 4d 0f c3 00 movnti %r8,\(%r8\) [ ]*[a-f0-9]+: 4c 0f c3 00 movnti %r8,\(%rax\) +[ ]*[a-f0-9]+: e2 fe loop 0x[0-9a-f]+ +[ ]*[a-f0-9]+: e2 fe loop 0x[0-9a-f]+ +[ ]*[a-f0-9]+: 67 e2 fd loopl 0x[0-9a-f]+ +[ ]*[a-f0-9]+: e3 fe jrcxz 0x[0-9a-f]+ +[ ]*[a-f0-9]+: 67 e3 fd jecxz 0x[0-9a-f]+ [ ]*[a-f0-9]+: 41 f6 38 idivb \(%r8\) [ ]*[a-f0-9]+: f6 38 idivb \(%rax\) [ ]*[a-f0-9]+: 66 41 f7 38 idivw \(%r8\) @@ -296,5 +301,4 @@ Disassembly of section .text: [ ]*[a-f0-9]+: 0f 07 sysret [ ]*[a-f0-9]+: 0f 01 f8 swapgs [ ]*[a-f0-9]+: 66 68 22 22 pushw \$0x2222 -[ ]*[a-f0-9]+: 67 e3 ff jecxz 0x49d #pass --- 2013-04-08/gas/testsuite/gas/i386/x86-64-opcode.s +++ 2013-04-08/gas/testsuite/gas/i386/x86-64-opcode.s @@ -61,15 +61,18 @@ # Conditionals # LOOP + LOOP . # -- -- -- -- E2 FE ; RCX used as counter. + LOOPq . # -- -- -- -- E2 FE ; RCX used as counter. + LOOPl . # -- 67 -- -- E2 FD ; ECX used as counter. # Jcc - # 66 -- -- -- 77 FD ; A16 override: (Addr64) = ZEXT(Addr16) - # 66 -- -- -- 0F 87 F9 FF FF FF ; A16 override: (Addr64) = ZEXT(Addr16) + # 66 -- -- -- 77 FD ; O16 override: (Addr64) = ZEXT(Addr16) + # 66 -- -- -- 0F 87 F9 FF FF FF ; O16 override: (Addr64) = ZEXT(Addr16) # J*CXZ - # 66 67 -- -- E3 FC ; ECX used as counter. A16 override: (Addr64) = ZEXT(Addr16) - # 66 -- -- -- E3 FD ; A16 override: (Addr64) = ZEXT(Addr16) + JRCXZ . # -- -- -- -- E3 FE ; RCX used as counter. + JECXZ . # -- 67 -- -- E3 FD ; ECX used as counter. @@ -424,5 +427,3 @@ swapgs # -- -- -- -- 0F 01 f8 pushw $0x2222 - - jecxz .+2 --- 2013-04-08/opcodes/i386-opc.tbl +++ 2013-04-08/opcodes/i386-opc.tbl @@ -389,8 +389,7 @@ jg, 1, 0x7f, None, 1, 0, Jump|No_bSuf|No // jcxz vs. jecxz is chosen on the basis of the address size prefix. jcxz, 1, 0xe3, None, 1, CpuNo64, JumpByte|Size16|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf, { Disp8|Disp16|Disp32 } -jecxz, 1, 0xe3, None, 1, CpuNo64, JumpByte|Size32|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf, { Disp8|Disp16|Disp32 } -jecxz, 1, 0xe3, None, 1, Cpu64, JumpByte|Size32|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf, { Disp8|Disp32|Disp32S } +jecxz, 1, 0xe3, None, 1, 0, JumpByte|Size32|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf, { Disp8|Disp16|Disp32|Disp32S } jrcxz, 1, 0xe3, None, 1, Cpu64, JumpByte|Size64|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|NoRex64, { Disp8|Disp32|Disp32S } // The loop instructions also use the address size prefix to select --- 2013-04-08/opcodes/i386-tbl.h +++ 2013-04-08/opcodes/i386-tbl.h @@ -3410,23 +3410,12 @@ const insn_template i386_optab[] = { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 1, 0 } }, - { 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 1, 1, - 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 }, - { { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0 } } } }, - { "jecxz", 1, 0xe3, None, 1, - { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 0, 1, 0, 0 } }, + 0, 0, 0, 0 } }, { 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 }, { { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, - 1, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 } } } }, { "jrcxz", 1, 0xe3, None, 1, { { 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,