[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