This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH i386 AVX512] [32/n] Add reduce,range,fpclass.
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Richard Henderson <rth at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 9 Sep 2014 14:20:20 +0200
- Subject: Re: [PATCH i386 AVX512] [32/n] Add reduce,range,fpclass.
- Authentication-results: sourceware.org; auth=none
- References: <20140829135525 dot GD18938 at msticlxl57 dot ims dot intel dot com> <CAFULd4bbZwoHEeYULu6q92K8VQTKpHLDOfM_aWxwfBxD6dp=6w at mail dot gmail dot com> <20140909114134 dot GB62820 at msticlxl57 dot ims dot intel dot com>
On Tue, Sep 9, 2014 at 1:41 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> On 30 Aug 10:21, Uros Bizjak wrote:
> Hello,
>> It looks to me that _SCALAR unspecs are redundant, and should be
>> possible to use UNSPEC_REDUCE for all patterns without unwanted
>> matching.
> Updated patch in the bottom
> gcc/
> * config/i386/i386.c
> (ix86_expand_args_builtin): Handle avx512dq_rangepv8df_mask_round,
> avx512dq_rangepv16sf_mask_round, avx512dq_rangepv4df_mask,
> avx512dq_rangepv8sf_mask, avx512dq_rangepv2df_mask,
> avx512dq_rangepv4sf_mask.
> * config/i386/sse.md
> (define_c_enum "unspec"): Add UNSPEC_REDUCE, UNSPEC_FPCLASS,
> UNSPEC_RANGE.
> (define_insn "<mask_codefor>reducep<mode><mask_name>"): New.
> (define_insn "reduces<mode>"): New.
> (define_insn "avx512dq_rangep<mode><mask_name><round_saeonly_name>"):
> New.
> (define_insn "avx512dq_ranges<mode><round_saeonly_name>"): New.
> (define_insn "avx512dq_fpclass<mode><mask_scalar_merge_name>"): New.
> (define_insn "avx512dq_vmfpclass<mode>"): New.
>
> Bootstrapped.
> Is it ok for trunk?
OK.
Thanks,
Uros.