[Bug gas/14859] Redundant ASP generated for jecxz

[hjl.tools at gmail dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=14859

--- Comment #4 from H.J. Lu <hjl.tools at gmail dot com> 2012-11-20 13:58:15 
UTC ---
(In reply to comment #1)
> 
> Possible fix would be:
> $ git diff opcodes/i386-opc.tbl
> diff --git a/opcodes/i386-opc.tbl b/opcodes/i386-opc.tbl
> index f2b2f14..e7001e9 100644
> --- a/opcodes/i386-opc.tbl
> +++ b/opcodes/i386-opc.tbl
> @@ -389,8 +389,7 @@ jg, 1, 0x7f, None, 1, 0,
> Jump|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf,
> 
>  // 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, 0x67e3, None, 2, 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 }
>  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

We shouldn't combine Disp32S with Disp32.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.