public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH][GRAPHITE] Make --param loop-block-tile-size=0 disable tiling
@ 2017-09-27 12:20 Richard Biener
  2017-09-28 20:45 ` Sebastian Pop
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Biener @ 2017-09-27 12:20 UTC (permalink / raw)
  To: gcc-patches


Currently ISL aborts on this special value and for debugging (and 
tuning?) it's nice to avoid all the clutter introduced by tiling.

Committed as obvious.

Richard.

2017-09-27  Richard Biener  <rguenther@suse.de>

	* graphite-optimize-isl.c (get_schedule_for_node_st): Allow
	--param loop-block-tile-size=0 to disable tiling.

Index: gcc/graphite-optimize-isl.c
===================================================================
--- gcc/graphite-optimize-isl.c	(revision 253226)
+++ gcc/graphite-optimize-isl.c	(working copy)
@@ -64,7 +64,10 @@ get_schedule_for_node_st (__isl_take isl
   if (type != isl_schedule_node_leaf)
     return node;
 
-  if (dims <= 1 || !isl_schedule_node_band_get_permutable (node))
+  long tile_size = PARAM_VALUE (PARAM_LOOP_BLOCK_TILE_SIZE);
+  if (dims <= 1
+      || tile_size == 0
+      || !isl_schedule_node_band_get_permutable (node))
     {
       if (dump_file && dump_flags)
 	fprintf (dump_file, "not tiled\n");
@@ -74,7 +77,6 @@ get_schedule_for_node_st (__isl_take isl
   /* Tile loops.  */
   space = isl_schedule_node_band_get_space (node);
   isl_multi_val *sizes = isl_multi_val_zero (space);
-  long tile_size = PARAM_VALUE (PARAM_LOOP_BLOCK_TILE_SIZE);
   isl_ctx *ctx = isl_schedule_node_get_ctx (node);
 
   for (unsigned i = 0; i < dims; i++)

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH][GRAPHITE] Make --param loop-block-tile-size=0 disable tiling
  2017-09-27 12:20 [PATCH][GRAPHITE] Make --param loop-block-tile-size=0 disable tiling Richard Biener
@ 2017-09-28 20:45 ` Sebastian Pop
  0 siblings, 0 replies; 2+ messages in thread
From: Sebastian Pop @ 2017-09-28 20:45 UTC (permalink / raw)
  To: Richard Biener; +Cc: GCC Patches

On Wed, Sep 27, 2017 at 7:20 AM, Richard Biener <rguenther@suse.de> wrote:
>
> Currently ISL aborts on this special value and for debugging (and
> tuning?) it's nice to avoid all the clutter introduced by tiling.
>
> Committed as obvious.
>
> Richard.
>
> 2017-09-27  Richard Biener  <rguenther@suse.de>
>
>         * graphite-optimize-isl.c (get_schedule_for_node_st): Allow
>         --param loop-block-tile-size=0 to disable tiling.

Looks good.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-09-28 20:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-27 12:20 [PATCH][GRAPHITE] Make --param loop-block-tile-size=0 disable tiling Richard Biener
2017-09-28 20:45 ` Sebastian Pop

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).