public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] x86: drop redundant EVex128 from PUSH2/POP2
@ 2024-01-12  8:58 Jan Beulich
  2024-01-18  1:35 ` Cui, Lili
  0 siblings, 1 reply; 2+ messages in thread
From: Jan Beulich @ 2024-01-12  8:58 UTC (permalink / raw)
  To: Binutils; +Cc: H.J. Lu, Mo, Zewei, Lili Cui

EVexMap4 already covers that.

--- a/opcodes/i386-opc.tbl
+++ b/opcodes/i386-opc.tbl
@@ -3494,9 +3494,9 @@ uwrmsr, 0xf3f8/0, APX_F(USER_MSR), Modrm
 
 // APX Push2/Pop2 instructions.
 
-push2, 0xff/6, APX_F, Modrm|VexW0|EVex128|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSuf, { Reg64, Reg64 }
-push2p, 0xff/6, APX_F, Modrm|VexW1|EVex128|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSuf, { Reg64, Reg64 }
-pop2, 0x8f/0, APX_F, Modrm|VexW0|EVex128|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSuf, { Reg64, Reg64 }
-pop2p, 0x8f/0, APX_F, Modrm|VexW1|EVex128|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSuf, { Reg64, Reg64 }
+push2, 0xff/6, APX_F, Modrm|VexW0|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSuf, { Reg64, Reg64 }
+push2p, 0xff/6, APX_F, Modrm|VexW1|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSuf, { Reg64, Reg64 }
+pop2, 0x8f/0, APX_F, Modrm|VexW0|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSuf, { Reg64, Reg64 }
+pop2p, 0x8f/0, APX_F, Modrm|VexW1|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSuf, { Reg64, Reg64 }
 
 // APX Push2/Pop2 instructions end.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [PATCH] x86: drop redundant EVex128 from PUSH2/POP2
  2024-01-12  8:58 [PATCH] x86: drop redundant EVex128 from PUSH2/POP2 Jan Beulich
@ 2024-01-18  1:35 ` Cui, Lili
  0 siblings, 0 replies; 2+ messages in thread
From: Cui, Lili @ 2024-01-18  1:35 UTC (permalink / raw)
  To: Beulich, Jan, Binutils; +Cc: H.J. Lu, Mo, Zewei

> EVexMap4 already covers that.
> 
> --- a/opcodes/i386-opc.tbl
> +++ b/opcodes/i386-opc.tbl
> @@ -3494,9 +3494,9 @@ uwrmsr, 0xf3f8/0, APX_F(USER_MSR), Modrm
> 
>  // APX Push2/Pop2 instructions.
> 
> -push2, 0xff/6, APX_F,
> Modrm|VexW0|EVex128|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf
> |No_sSuf, { Reg64, Reg64 } -push2p, 0xff/6, APX_F,
> Modrm|VexW1|EVex128|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf
> |No_sSuf, { Reg64, Reg64 } -pop2, 0x8f/0, APX_F,
> Modrm|VexW0|EVex128|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf
> |No_sSuf, { Reg64, Reg64 } -pop2p, 0x8f/0, APX_F,
> Modrm|VexW1|EVex128|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf
> |No_sSuf, { Reg64, Reg64 }
> +push2, 0xff/6, APX_F,
> +Modrm|VexW0|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSu
> f, { Reg64,
> +Reg64 } push2p, 0xff/6, APX_F,
> +Modrm|VexW1|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSu
> f, { Reg64,
> +Reg64 } pop2, 0x8f/0, APX_F,
> +Modrm|VexW0|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSu
> f, { Reg64,
> +Reg64 } pop2p, 0x8f/0, APX_F,
> +Modrm|VexW1|EVexMap4|VexVVVV|No_bSuf|No_wSuf|No_lSuf|No_sSu
> f, { Reg64,
> +Reg64 }
> 
>  // APX Push2/Pop2 instructions end.

It's OK and thanks.

Lili.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-01-18  1:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-12  8:58 [PATCH] x86: drop redundant EVex128 from PUSH2/POP2 Jan Beulich
2024-01-18  1:35 ` Cui, Lili

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).