This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [RFT PATCH, AVX512]: Implement scalar float->unsigned int truncations with AVX512F
- From: "Peryt, Sebastian" <sebastian dot peryt at intel dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Kirill Yukhin <kirill dot yukhin at gmail dot com>, "Peryt, Sebastian" <sebastian dot peryt at intel dot com>
- Date: Wed, 23 May 2018 07:55:47 +0000
- Subject: RE: [RFT PATCH, AVX512]: Implement scalar float->unsigned int truncations with AVX512F
- Dlp-product: dlpe-windows
- Dlp-reaction: no-action
- Dlp-version: 11.0.200.100
- References: <CAFULd4abHKLNKwmK1jQ7iSUr9Fnn_6ReLh39X-FwjA_zh48NLA@mail.gmail.com> <CAFULd4YAmF1HF73jjs5cpnYUO_Ko57LqZJ2=W74ZMGRKypF72A@mail.gmail.com>
> From: gcc-patches-owner@gcc.gnu.org [mailto:gcc-patches-
> owner@gcc.gnu.org] On Behalf Of Uros Bizjak
> Sent: Monday, May 21, 2018 9:55 PM
> To: gcc-patches@gcc.gnu.org
> Cc: Jakub Jelinek <jakub@redhat.com>; Kirill Yukhin
> <kirill.yukhin@gmail.com>
> Subject: Re: [RFT PATCH, AVX512]: Implement scalar float->unsigned int
> truncations with AVX512F
>
> On Mon, May 21, 2018 at 4:53 PM, Uros Bizjak <ubizjak@gmail.com> wrote:
> > Hello!
> >
> > Attached patch implements scalar float->unsigned int truncations
> > with
> AVX512F.
> >
> > 2018-05-21 Uros Bizjak <ubizjak@gmail.com>
> >
> > * config/i386/i386.md (fixuns_trunc<mode>di2): New insn pattern.
> > (fixuns_trunc<mode>si2_avx512f): Ditto.
> > (*fixuns_trunc<mode>si2_avx512f_zext): Ditto.
> > (fixuns_trunc<mode>si2): Also enable for AVX512F and TARGET_SSE_MATH.
> > Emit fixuns_trunc<mode>si2_avx512f for AVX512F targets.
> >
> > testsuite/ChangeLog:
> >
> > 2018-05-21 Uros Bizjak <ubizjak@gmail.com>
> >
> > * gcc.target/i386/cvt-2.c: New test.
> >
> > Patch was bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.
> >
> > Unfortunately, I have to means to test the patch on AVX512 target,
> > so to avoid some hidden issue, I'd like to ask someone to test it on
> > live target.
I've bootstrapped and regression tested your patch on x86_64-linux-gnu {,-m32} on SKX machine and I don't see any stability regression.
Sebastian
>
> Ops, ssemodesuffix handling was missing in the insn mnemonic. Fixed in
> the attached v-2 patch.
>
> Uros.