[PATCH] x86: have .insn correctly consider AVX10.2's 256-bit embedded rounding

Jiang, Haochen haochen.jiang@intel.com
Fri Jan 17 02:21:22 GMT 2025


> From: Jan Beulich <jbeulich@suse.com>
> Sent: Thursday, January 16, 2025 5:17 PM
> 
> On 16.01.2025 10:12, Jiang, Haochen wrote:
> >> From: Jan Beulich <jbeulich@suse.com>
> >> Sent: Thursday, January 16, 2025 5:10 PM
> >>
> >> Deriving operand size may no longer assume 512-bit vector size when
> >> embedded rounding is in use. In fact it was apparently wrong to do so
> >> in the first place, as that's not correct for scalar insns. Drop the
> >> rounding type check altogether; we fall back to EVEX.LIG when no
> >> suitable operand was specified anyway, later in the function (and,
> >> btw, similarly for VEX encodings).
> >> ---
> >> Testcase to be the one from "x86: Ignore rounding for vcvt[,u]si2sd
> >> under r32 and vcvt[,u]dq2pd instead of reporting bad", with vector
> >> size zapped from the input.
> >
> > LGTM since texi actually needs no change after another look at that.
> >
> > I will adjust my testcase afterwards.
> 
> Actually I'd like your change to go in first, for mine to then adjust the testcase
> accordingly.
> 

Patch committed with only disassembler change.

Thx,
Haochen


More information about the Binutils mailing list