From patchwork Fri Jul 16 07:04:00 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [09/12] Add testcase for PR42729 and fix handling of gimple_debug info. Date: Thu, 15 Jul 2010 21:04:00 -0000 From: Sebastian Pop X-Patchwork-Id: 59077 Message-Id: <1279263843-9149-10-git-send-email-sebpop@gmail.com> To: gcc-patches@gcc.gnu.org Cc: gcc-graphite@googlegroups.com, Sebastian Pop 2010-07-15 Sebastian Pop * sese.c (rename_uses): Handl unconditionally gimple_debug statements. * gcc.dg/graphite/pr42729.c: New. --- gcc/ChangeLog.graphite | 6 ++++++ gcc/sese.c | 24 +++++++++++------------- gcc/testsuite/gcc.dg/graphite/pr42729.c | 15 +++++++++++++++ 3 files changed, 32 insertions(+), 13 deletions(-) create mode 100644 gcc/testsuite/gcc.dg/graphite/pr42729.c diff --git a/gcc/ChangeLog.graphite b/gcc/ChangeLog.graphite index 40db483..f57ef7d 100644 --- a/gcc/ChangeLog.graphite +++ b/gcc/ChangeLog.graphite @@ -1,5 +1,11 @@ 2010-07-15 Sebastian Pop + * sese.c (rename_uses): Handl unconditionally gimple_debug statements. + + * gcc.dg/graphite/pr42729.c: New. + +2010-07-15 Sebastian Pop + * graphite-sese-to-poly.c (rewrite_close_phi_out_of_ssa): Special case non close-phi nodes with one argument. diff --git a/gcc/sese.c b/gcc/sese.c index 913ac4d..f59f317 100644 --- a/gcc/sese.c +++ b/gcc/sese.c @@ -480,6 +480,16 @@ rename_uses (gimple copy, htab_t rename_map, gimple_stmt_iterator *gsi_tgt, use_operand_p use_p; ssa_op_iter op_iter; + if (is_gimple_debug (copy)) + { + if (gimple_debug_bind_p (copy)) + gimple_debug_bind_reset_value (copy); + else + gcc_unreachable (); + + return; + } + FOR_EACH_SSA_USE_OPERAND (use_p, copy, op_iter, SSA_OP_ALL_USES) { tree old_name = USE_FROM_PTR (use_p); @@ -501,19 +511,7 @@ rename_uses (gimple copy, htab_t rename_map, gimple_stmt_iterator *gsi_tgt, || (TREE_CODE (new_expr) != SSA_NAME && is_gimple_reg (old_name))) { - tree var; - - if (is_gimple_debug (copy)) - { - if (gimple_debug_bind_p (copy)) - gimple_debug_bind_reset_value (copy); - else - gcc_unreachable (); - - break; - } - - var = create_tmp_var (type_old_name, "var"); + tree var = create_tmp_var (type_old_name, "var"); if (type_old_name != type_new_expr) new_expr = fold_convert (type_old_name, new_expr); diff --git a/gcc/testsuite/gcc.dg/graphite/pr42729.c b/gcc/testsuite/gcc.dg/graphite/pr42729.c new file mode 100644 index 0000000..3a0e901 --- /dev/null +++ b/gcc/testsuite/gcc.dg/graphite/pr42729.c @@ -0,0 +1,15 @@ +/* { dg-do compile } */ +/* { dg-options "-O1 -fgraphite-identity -fcompare-debug" } */ + +int A[10]; +int *foo () +{ + int *p1, *p2, i; + for (i = 0; i < 10; i++) + { + p1 = &A[i]; + *p1 = 0; + } + p2 = p1; + return p2; +}