[PATCH 1/2] [PATCH 1/2] Enable Intel AVX512_FP16 instructions

Jan Beulich jbeulich@suse.com
Tue Jul 13 16:25:03 GMT 2021


On 13.07.2021 08:58, Cui, Lili wrote:

Assembler:

In check_word_reg() the comment will want to say "EVEX.W" instead of
"REX byte".

I don't think you've applied my comments regarding the cpu_flag_init[]
additions.

I can't seem to find any pseudos of VCMP{P,S}H, despite the prior
comment.

There are two VMOVW templates now, but the first still allows for both
Reg32 and Word, effectively being Reg32|Reg16.

There's an IgnoreSize on one of the VCVTPH2PSX, but none on the operand-
size-wise similar VCVTPH2{,U}DQ. I can't right away tell which one is
right, but this set of templates needs to be consistent in this regard.

Jan



More information about the Binutils mailing list