This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFC, PR66873] Use graphite for parloops
- From: Sebastian Pop <sebpop at gmail dot com>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: "gcc-patches at gnu dot org" <gcc-patches at gnu dot org>
- Date: Tue, 21 Jul 2015 18:42:49 +0000
- Subject: Re: [RFC, PR66873] Use graphite for parloops
- Authentication-results: sourceware.org; auth=none
- References: <55A6C1DF dot 1050108 at mentor dot com> <20150720183141 dot GB20717 at f1 dot c dot bardezibar dot internal> <55AD9093 dot 1060206 at mentor dot com> <55AE5340 dot 2010700 at mentor dot com>
Tom de Vries wrote:
> Fix reduction safety checks
>
> * graphite-sese-to-poly.c (is_reduction_operation_p): Limit
> flag_associative_math to SCALAR_FLOAT_TYPE_P. Honour
> TYPE_OVERFLOW_TRAPS and TYPE_OVERFLOW_WRAPS for INTEGRAL_TYPE_P.
> Only allow wrapping fixed-point otherwise.
> (build_poly_scop): Always call
> rewrite_commutative_reductions_out_of_ssa.
The changes to graphite look good to me.
Thanks,
Sebastian