This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] Fix PR42771: Rename SESE parameters in all SCoPs after Graphite code gen
- From: Alexander Monakov <amonakov at ispras dot ru>
- To: Sebastian Pop <sebpop at gmail dot com>
- Cc: gcc-graphite <gcc-graphite at googlegroups dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 10 Feb 2010 21:37:06 +0300 (MSK)
- Subject: Re: [patch] Fix PR42771: Rename SESE parameters in all SCoPs after Graphite code gen
- References: <cb9d34b21002100900o3e3a54cdge3a49875fad7cb88@mail.gmail.com>
Hi,
diff --git a/gcc/graphite.c b/gcc/graphite.c
index 3c8c4ce..d28b9cf 100644
--- a/gcc/graphite.c
+++ b/gcc/graphite.c
@@ -269,20 +269,13 @@ graphite_transform_loops (void)
bb_pbb_mapping = htab_create (10, bb_pbb_map_hash, eq_bb_pbb_map, free);
for (i = 0; VEC_iterate (scop_p, scops, i, scop); i++)
- {
- bool transform_done = false;
-
- if (!dbg_cnt (graphite_scop) || !build_poly_scop (scop))
- continue;
+ build_poly_scop (scop);
- if (apply_poly_transforms (scop))
- transform_done = gloog (scop, bb_pbb_mapping);
- else
- check_poly_representation (scop);
-
- if (transform_done)
- need_cfg_cleanup_p = true;
- }
+ for (i = 0; VEC_iterate (scop_p, scops, i, scop); i++)
+ if (POLY_SCOP_P (scop)
+ && apply_poly_transforms (scop)
+ && gloog (scop, scops, bb_pbb_mapping))
+ need_cfg_cleanup_p = true;
htab_delete (bb_pbb_mapping);
free_scops (scops);
This drops dbg_cnt and check_poly_representation calls, is that intentional?
I think dbg_cnt call should look like:
for (i = 0; VEC_iterate (scop_p, scops, i, scop); i++)
if (dbg_cnt (graphite_scop))
build_poly_scop (scop);
Thanks.
Alexander Monakov