> From: Jiang, Haochen
> Sent: Monday, December 22, 2025 10:05 AM
> 
> > From: Gerald Pfeifer <[email protected]>
> > Sent: Saturday, December 20, 2025 8:14 AM
> >
> > On Mon, 15 Dec 2025, Haochen Jiang wrote:
> >
> > Maybe add "any longer" at the end?
> >
> > > +    Based on ISA extensions enabled on Alder Lake, the switch intially
> further
> > > +    enables the AVX-IFMA, AVX-NE-CONVERT, AVX-VNNI-INT8 and
> CMPccXADD ISA
> > > +    extensions. Since GCC 13.2, ENQCMD and UINTR are further enabled.
> >
> > Typo, "initially".
> >
> > Though how about saying "this switch in addition enables ... Since GCC
> > 13.2, ENQCMD and UINTR are enabled as well"?
> 
> Should I also unify the wording to "in addition" from "further" in other
> scenario, i.e., other march options?

I have changed all of them to "in addition" and applied the patch

Thx,
Haochen

> 
> >
> > > +  <li><code>-mavx10.1-256</code>, <code>-mavx10.1-512</code>, and
> > > +    <code>-mevex512</code> were removed together with the warning for
> the
> > > +    behavior change on <code>-mavx10.1</code>. <code>-
> mavx10.1</code> has
> > > +    enabled AVX10.1 intrinsics with 512-bit vector support since GCC 15.
> >
> > Just to make sure: the warning was removed? If so, all good.
> 
> Yes they were removed.
> 
> >
> > Wow, that's quite a bit. Thanks for your careful work tracking all these!
> >
> > (Patch is okay.)
> 
> I will apply all the suggestions in the thread.
> 
> Thx,
> Haochen

Reply via email to