[PATCH] x86: drop vex_encoding_vex2 enumerator

Jan Beulich jbeulich@suse.com
Fri Jan 17 14:57:00 GMT 2020


On 17.01.2020 15:48, H.J. Lu wrote:
> On Fri, Jan 17, 2020 at 1:03 AM Jan Beulich <jbeulich@suse.com> wrote:
>>
>> On 16.01.2020 23:32,  H.J. Lu  wrote:
>>> How about this?
>>
>> For {vex2} "if possible" still leaves open what happens if this
>> is not possible. Aiui there are two distinct cases:
>> - 3-byte VEX could be used, in which case the prefix is ignored
> 
> Did you mean "must be used"?

Well, yes, in a way.

>> - only legacy or EVEX encoding is possible, in which case there'd
>>   be an error (not really sure about EVEX here)
>> I think it would be better to have {vex} and {vex3}.
> 
> Agreed.  I will make the change and keep {vex2} for backward compatibility.
> How about this patch?
> 
>> Furthermore both say "for VEX instruction", leaving open what is
>> to happen for non-VEX-encodable ones. It also leaves open what
>> their interaction with XOP-encoded insns is.
> 
> Here is the updated document.

Looks good, thanks.

Jan



More information about the Binutils mailing list