[PATCH] Enable reductions without fassociative-math in graphite

[PATCH] Enable reductions without fassociative-math in graphite

[Tom de Vries]

[-- Attachment #1: Type: text/plain, Size: 250 bytes --]

Hi,

this patch allows non-float reductions to be detected by graphite, 
independent of whether fassociative-math (which only has effect for 
float operations) is set.

Currently bootstrapping and reg-testing on x86_64.

OK for trunk?

Thanks,
- Tom

[-- Attachment #2: 0002-Enable-reductions-without-fassociative-math-in-graph.patch --]
[-- Type: text/x-patch, Size: 808 bytes --]

Enable reductions without fassociative-math in graphite

2015-07-21  Tom de Vries  <tom@codesourcery.com>

	* graphite-sese-to-poly.c (build_poly_scop): Always call
	rewrite_commutative_reductions_out_of_ssa.
---
 gcc/graphite-sese-to-poly.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gcc/graphite-sese-to-poly.c b/gcc/graphite-sese-to-poly.c
index 68f7df1..28b9817 100644
--- a/gcc/graphite-sese-to-poly.c
+++ b/gcc/graphite-sese-to-poly.c
@@ -3155,8 +3155,7 @@ build_poly_scop (scop_p scop)
   if (!scop_ivs_can_be_represented (scop))
     return;
 
-  if (flag_associative_math)
-    rewrite_commutative_reductions_out_of_ssa (scop);
+  rewrite_commutative_reductions_out_of_ssa (scop);
 
   build_sese_loop_nests (region);
   /* Record all conditions in REGION.  */
-- 
1.9.1

Re: [PATCH] Enable reductions without fassociative-math in graphite

[Richard Biener]

On Wed, 22 Jul 2015, Tom de Vries wrote:

> Hi,
> 
> this patch allows non-float reductions to be detected by graphite, independent
> of whether fassociative-math (which only has effect for float operations) is
> set.
> 
> Currently bootstrapping and reg-testing on x86_64.
> 
> OK for trunk?

Ok

> Thanks,
> - Tom
> 

-- 
Richard Biener <rguenther@suse.de>
SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Dilip Upmanyu, Graham Norton, HRB 21284 (AG Nuernberg)