diff options
author | Sebastian Pop <s.pop@samsung.com> | 2015-07-25 09:22:29 +0000 |
---|---|---|
committer | Sebastian Pop <spop@gcc.gnu.org> | 2015-07-25 09:22:29 +0000 |
commit | 33df361a844c44f23ad435960e163ab1732b6d2a (patch) | |
tree | b75d258502a2fdfede52aee8f6116fa73856a4fb /gcc | |
parent | 4c141304a869524c909a6a9346b004cda47659fc (diff) | |
download | gcc-33df361a844c44f23ad435960e163ab1732b6d2a.zip gcc-33df361a844c44f23ad435960e163ab1732b6d2a.tar.gz gcc-33df361a844c44f23ad435960e163ab1732b6d2a.tar.bz2 |
dump reasons why graphite failed to detect a scop
When trying to analyze why Graphite does not handle a loop nest, it is easy to
look in the dumps of -fdump-tree-graphite-all to guess what has to be changed to
catch the loop. This patch makes the dumps a bit more verbose and useful.
* graphite-scop-detection.c: Include gimple-pretty-print.h.
(stmt_simple_for_scop_p): Print when a stmt is not handled in Graphite.
(scopdet_basic_block_info): Print when a loop or bb cannot be
represented in Graphite.
From-SVN: r226213
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 8 | ||||
-rw-r--r-- | gcc/graphite-scop-detection.c | 72 |
2 files changed, 74 insertions, 6 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 60c4ade..04f7fab 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,11 @@ +2015-07-25 Sebastian Pop <s.pop@samsung.com> + + * graphite-scop-detection.c: Include gimple-pretty-print.h. + (stmt_simple_for_scop_p): Print when a stmt is not handled in + Graphite. + (scopdet_basic_block_info): Print when a loop or bb cannot be + represented in Graphite. + 2015-07-25 Uros Bizjak <ubizjak@gmail.com> PR target/66648 diff --git a/gcc/graphite-scop-detection.c b/gcc/graphite-scop-detection.c index 686f495..fb7247e 100644 --- a/gcc/graphite-scop-detection.c +++ b/gcc/graphite-scop-detection.c @@ -51,6 +51,7 @@ along with GCC; see the file COPYING3. If not see #include "graphite-poly.h" #include "tree-ssa-propagate.h" #include "graphite-scop-detection.h" +#include "gimple-pretty-print.h" /* Forward declarations. */ static void make_close_phi_nodes_unique (basic_block); @@ -350,13 +351,31 @@ stmt_simple_for_scop_p (basic_block scop_entry, loop_p outermost_loop, || (gimple_code (stmt) == GIMPLE_CALL && !(gimple_call_flags (stmt) & (ECF_CONST | ECF_PURE))) || (gimple_code (stmt) == GIMPLE_ASM)) - return false; + { + if (dump_file && (dump_flags & TDF_DETAILS)) + { + fprintf (dump_file, "[scop-detection-fail] "); + fprintf (dump_file, "Graphite cannot handle this stmt:\n"); + print_gimple_stmt (dump_file, stmt, 0, TDF_VOPS|TDF_MEMSYMS); + } + + return false; + } if (is_gimple_debug (stmt)) return true; if (!stmt_has_simple_data_refs_p (outermost_loop, stmt)) - return false; + { + if (dump_file && (dump_flags & TDF_DETAILS)) + { + fprintf (dump_file, "[scop-detection-fail] "); + fprintf (dump_file, "Graphite cannot handle data-refs in stmt:\n"); + print_gimple_stmt (dump_file, stmt, 0, TDF_VOPS|TDF_MEMSYMS); + } + + return false; + } switch (gimple_code (stmt)) { @@ -375,7 +394,16 @@ stmt_simple_for_scop_p (basic_block scop_entry, loop_p outermost_loop, || code == GE_EXPR || code == EQ_EXPR || code == NE_EXPR)) - return false; + { + if (dump_file && (dump_flags & TDF_DETAILS)) + { + fprintf (dump_file, "[scop-detection-fail] "); + fprintf (dump_file, "Graphite cannot handle cond stmt:\n"); + print_gimple_stmt (dump_file, stmt, 0, TDF_VOPS|TDF_MEMSYMS); + } + + return false; + } for (unsigned i = 0; i < 2; ++i) { @@ -383,7 +411,16 @@ stmt_simple_for_scop_p (basic_block scop_entry, loop_p outermost_loop, if (!graphite_can_represent_expr (scop_entry, loop, op) /* We can not handle REAL_TYPE. Failed for pr39260. */ || TREE_CODE (TREE_TYPE (op)) == REAL_TYPE) - return false; + { + if (dump_file && (dump_flags & TDF_DETAILS)) + { + fprintf (dump_file, "[scop-detection-fail] "); + fprintf (dump_file, "Graphite cannot represent stmt:\n"); + print_gimple_stmt (dump_file, stmt, 0, TDF_VOPS|TDF_MEMSYMS); + } + + return false; + } } return true; @@ -395,6 +432,12 @@ stmt_simple_for_scop_p (basic_block scop_entry, loop_p outermost_loop, default: /* These nodes cut a new scope. */ + if (dump_file && (dump_flags & TDF_DETAILS)) + { + fprintf (dump_file, "[scop-detection-fail] "); + fprintf (dump_file, "Gimple stmt not handled in Graphite:\n"); + print_gimple_stmt (dump_file, stmt, 0, TDF_VOPS|TDF_MEMSYMS); + } return false; } @@ -488,7 +531,16 @@ scopdet_basic_block_info (basic_block bb, loop_p outermost_loop, with make_forwarder_block. */ if (!single_succ_p (bb) || bb_has_abnormal_pred (single_succ (bb))) - result.difficult = true; + { + if (dump_file && (dump_flags & TDF_DETAILS)) + { + fprintf (dump_file, "[scop-detection-fail] "); + fprintf (dump_file, "BB %d cannot be part of a scop.\n", + bb->index); + } + + result.difficult = true; + } else result.exit = single_succ (bb); @@ -509,7 +561,15 @@ scopdet_basic_block_info (basic_block bb, loop_p outermost_loop, sinfo = build_scops_1 (bb, outermost_loop, ®ions, loop); if (!graphite_can_represent_loop (entry_block, loop)) - result.difficult = true; + { + if (dump_file && (dump_flags & TDF_DETAILS)) + { + fprintf (dump_file, "[scop-detection-fail] "); + fprintf (dump_file, "Graphite cannot represent loop %d.\n", + loop->num); + } + result.difficult = true; + } result.difficult |= sinfo.difficult; |