[PATCH v3] x86: Optimize load of const all 1s FP vectors

H.J. Lu hjl.tools@gmail.com
Mon Aug 9 17:46:34 GMT 2021


On Mon, Aug 9, 2021 at 8:27 AM Uros Bizjak <ubizjak@gmail.com> wrote:
>
> On Mon, Aug 9, 2021 at 5:24 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> >
> > On Sun, Aug 8, 2021 at 1:23 PM Uros Bizjak <ubizjak@gmail.com> wrote:
> > >
> > > On Sat, Aug 7, 2021 at 4:41 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > > >
> > > > Update vector_all_ones_operand to return true for const all 1s float
> > > > vectors.
> > > >
> > > > gcc/
> > > >
> > > >         PR target/101804
> > > >         * config/i386/predicates.md (vector_all_ones_operand): Return
> > > >         true for const all 1s float vectors.
> > > >
> > > > gcc/testsuite/
> > > >
> > > >         PR target/101804
> > > >         * gcc.target/i386/avx2-gather-2.c: Pass -march=skylake instead
> > > >         of "-mavx2 -mtune=skylake".  Scan vpcmpeqd.
> > >
> > > No, vector_all_ones_operand is intended to be integer minus-one. Use
> > > float_vector_all_ones_operand in a specific place, where it is needed.
> > >
> >
> > Like this?
>
> Please also add a new constraint, BC is intended for integer values.
>
> Uros.

Here is the v3 patch with the new BF constraint.  OK for master?

Thanks.

-- 
H.J.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: v3-0001-x86-Optimize-load-of-const-all-1s-FP-vectors.patch
Type: text/x-patch
Size: 5244 bytes
Desc: not available
URL: <https://gcc.gnu.org/pipermail/gcc-patches/attachments/20210809/76c81e41/attachment.bin>


More information about the Gcc-patches mailing list