[PATCH V2 3/8] Support APX GPR32 with extend evex prefix

[Jan Beulich]

On 09.11.2023 09:38, Cui, Lili wrote:
>> Subject: Re: [PATCH V2 3/8] Support APX GPR32 with extend evex prefix
>>
>> On 03.11.2023 17:50, Cui, Lili wrote:
>>> --- a/gas/config/tc-i386.c
>>> +++ b/gas/config/tc-i386.c
>>> @@ -3672,9 +3672,10 @@ install_template (const insn_template *t)
>>
>> What I'm surprised by is that you don't have any change to
>> cpu_flags_match().
>> I don't think you can get away without for dual VEX/EVEX templates. I hope
>> further down you'll find a sufficient explanation of what I think is going to be
>> needed.
>>
> 
> I think it is ok. Or am I missing something?
> 
>  $ cat a.s
>        .allow_index_reg
>         .text
> _start:
>         .text
>         .arch .noapx_f
> ldtilecfg  (%r31,%rdx,2)
>         .arch .noamx_tile
> ldtilecfg  (%rax,%rdx,2)
> 
> $ as --64 a.s -o a.o
> a.s: Assembler messages:
> a.s:9: Error: `ldtilecfg' is not supported on `x86_64.noapx_f'

Hmm, the error here ought to be about %r31 not being a valid register name.
In any event prior to .noamx_tile "is not supported" is wrong: The
instruction itself is valid, it's merely used with the wrong operand. What
you also want to check is "{evex} ldtilecfg (%rax,%rdx,2)".

> a.s:11: Error: `ldtilecfg' is not supported on `x86_64.noapx_f.noamx_tile'

This error message is correct.

Jan