x86: drop redundant EVex128 from PUSH2/POP2

Message ID c39fa6ba-927c-4828-bb65-69bc8a774535@suse.com
State Unresolved
Headers
Series x86: drop redundant EVex128 from PUSH2/POP2 |

Checks

Context Check Description
snail/binutils-gdb-check warning Git am fail log

Commit Message

Jan Beulich Jan. 12, 2024, 8:58 a.m. UTC
  EVexMap4 already covers that.
  

Comments

Cui, Lili Jan. 18, 2024, 1:35 a.m. UTC | #1
> 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.
  

Patch

--- 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.