OSDN Git Service

2010-08-04 Tobias Burnus <burnus@net-b.de>
[pf3gnuchains/gcc-fork.git] / gcc / graphite-interchange.c
index b42a133..0a7a749 100644 (file)
@@ -78,15 +78,15 @@ build_linearized_memory_access (ppl_dimension_type offset, poly_dr_p pdr)
   ppl_dimension_type i;
   ppl_dimension_type first = pdr_subscript_dim (pdr, 0);
   ppl_dimension_type last = pdr_subscript_dim (pdr, PDR_NB_SUBSCRIPTS (pdr));
-  Value size, sub_size;
+  mpz_t size, sub_size;
   graphite_dim_t dim = offset + pdr_dim (pdr);
 
   ppl_new_Linear_Expression_with_dimension (&res, dim);
 
-  value_init (size);
-  value_set_si (size, 1);
-  value_init (sub_size);
-  value_set_si (sub_size, 1);
+  mpz_init (size);
+  mpz_set_si (size, 1);
+  mpz_init (sub_size);
+  mpz_set_si (sub_size, 1);
 
   for (i = last - 1; i >= first; i--)
     {
@@ -95,21 +95,110 @@ build_linearized_memory_access (ppl_dimension_type offset, poly_dr_p pdr)
       ppl_new_Linear_Expression_with_dimension (&le, dim - offset);
       ppl_set_coef (le, i, 1);
       ppl_max_for_le_pointset (PDR_ACCESSES (pdr), le, sub_size);
-      value_multiply (size, size, sub_size);
+      mpz_mul (size, size, sub_size);
       ppl_delete_Linear_Expression (le);
     }
 
-  value_clear (sub_size);
-  value_clear (size);
+  mpz_clear (sub_size);
+  mpz_clear (size);
   return res;
 }
 
+/* Builds a partial difference equations and inserts them
+   into pointset powerset polyhedron P.  Polyhedron is assumed
+   to have the format: T|I|T'|I'|G|S|S'|l1|l2.
+
+   TIME_DEPTH is the time dimension w.r.t. which we are
+   differentiating.
+   OFFSET represents the number of dimensions between
+   columns t_{time_depth} and t'_{time_depth}.
+   DIM_SCTR is the number of scattering dimensions.  It is
+   essentially the dimensionality of the T vector.
+
+   The following equations are inserted into the polyhedron P:
+    | t_1 = t_1'
+    | ...
+    | t_{time_depth-1} = t'_{time_depth-1}
+    | t_{time_depth} = t'_{time_depth} + 1
+    | t_{time_depth+1} = t'_{time_depth + 1}
+    | ...
+    | t_{dim_sctr} = t'_{dim_sctr}.  */
+
+static void
+build_partial_difference (ppl_Pointset_Powerset_C_Polyhedron_t *p,
+                          ppl_dimension_type time_depth,
+                          ppl_dimension_type offset,
+                          ppl_dimension_type dim_sctr)
+{
+  ppl_Constraint_t new_cstr;
+  ppl_Linear_Expression_t le;
+  ppl_dimension_type i;
+  ppl_dimension_type dim;
+  ppl_Pointset_Powerset_C_Polyhedron_t temp;
+
+  /* Add the equality: t_{time_depth} = t'_{time_depth} + 1.
+     This is the core part of this alogrithm, since this
+     constraint asks for the memory access stride (difference)
+     between two consecutive points in time dimensions.  */
+
+  ppl_Pointset_Powerset_C_Polyhedron_space_dimension (*p, &dim);
+  ppl_new_Linear_Expression_with_dimension (&le, dim);
+  ppl_set_coef (le, time_depth, 1);
+  ppl_set_coef (le, time_depth + offset, -1);
+  ppl_set_inhomogeneous (le, 1);
+  ppl_new_Constraint (&new_cstr, le, PPL_CONSTRAINT_TYPE_EQUAL);
+  ppl_Pointset_Powerset_C_Polyhedron_add_constraint (*p, new_cstr);
+  ppl_delete_Linear_Expression (le);
+  ppl_delete_Constraint (new_cstr);
+
+  /* Add equalities:
+     | t1 = t1'
+     | ...
+     | t_{time_depth-1} = t'_{time_depth-1}
+     | t_{time_depth+1} = t'_{time_depth+1}
+     | ...
+     | t_{dim_sctr} = t'_{dim_sctr}
+
+     This means that all the time dimensions are equal except for
+     time_depth, where the constraint is t_{depth} = t'_{depth} + 1
+     step.  More to this: we should be carefull not to add equalities
+     to the 'coupled' dimensions, which happens when the one dimension
+     is stripmined dimension, and the other dimension corresponds
+     to the point loop inside stripmined dimension.  */
+
+  ppl_new_Pointset_Powerset_C_Polyhedron_from_Pointset_Powerset_C_Polyhedron (&temp, *p);
+
+  for (i = 0; i < dim_sctr; i++)
+    if (i != time_depth)
+      {
+        ppl_new_Linear_Expression_with_dimension (&le, dim);
+        ppl_set_coef (le, i, 1);
+        ppl_set_coef (le, i + offset, -1);
+        ppl_new_Constraint (&new_cstr, le, PPL_CONSTRAINT_TYPE_EQUAL);
+        ppl_Pointset_Powerset_C_Polyhedron_add_constraint (temp, new_cstr);
+
+        if (ppl_Pointset_Powerset_C_Polyhedron_is_empty (temp))
+          {
+            ppl_delete_Pointset_Powerset_C_Polyhedron (temp);
+            ppl_new_Pointset_Powerset_C_Polyhedron_from_Pointset_Powerset_C_Polyhedron (&temp, *p);
+          }
+        else
+          ppl_Pointset_Powerset_C_Polyhedron_add_constraint (*p, new_cstr);
+        ppl_delete_Linear_Expression (le);
+        ppl_delete_Constraint (new_cstr);
+      }
+
+  ppl_delete_Pointset_Powerset_C_Polyhedron (temp);
+}
+
+
 /* Set STRIDE to the stride of PDR in memory by advancing by one in
-   time dimension DEPTH.  */
+   the loop at DEPTH.  */
 
 static void
-memory_stride_in_loop (Value stride, graphite_dim_t depth, poly_dr_p pdr)
+pdr_stride_in_loop (mpz_t stride, graphite_dim_t depth, poly_dr_p pdr)
 {
+  ppl_dimension_type time_depth;
   ppl_Linear_Expression_t le, lma;
   ppl_Constraint_t new_cstr;
   ppl_dimension_type i, *map;
@@ -167,9 +256,11 @@ memory_stride_in_loop (Value stride, graphite_dim_t depth, poly_dr_p pdr)
     ppl_set_coef (lma, dim_L1, -1);
     ppl_new_Constraint (&new_cstr, lma, PPL_CONSTRAINT_TYPE_EQUAL);
     ppl_Pointset_Powerset_C_Polyhedron_add_constraint (p1, new_cstr);
+    ppl_delete_Linear_Expression (lma);
+    ppl_delete_Constraint (new_cstr);
   }
 
-  /* Now intersect all the parts to get:
+  /* Now intersect all the parts to get the polyhedron P1:
      T|I|0|0|G|0|0|0 |0
      0|I|0|0|G|S|0|0 |0
      0|0|0|0|0|S|0|l1|0
@@ -196,14 +287,14 @@ memory_stride_in_loop (Value stride, graphite_dim_t depth, poly_dr_p pdr)
 
     map = ppl_new_id_map (new_dim);
 
-    /* T->T' and I->I'.  */
+    /* TI -> T'I'.  */
     for (i = 0; i < offset; i++)
       ppl_interchange (map, i, i + offset);
 
-    /* l1->l2.  */
+    /* l1 -> l2.  */
     ppl_interchange (map, dim_L1, dim_L2);
 
-    /* S->S'.  */
+    /* S -> S'.  */
     for (i = 0; i < nb_subscripts; i++)
       ppl_interchange (map, offset + offsetg + i,
                       offset + offsetg + nb_subscripts + i);
@@ -212,49 +303,11 @@ memory_stride_in_loop (Value stride, graphite_dim_t depth, poly_dr_p pdr)
     free (map);
   }
 
-  /* Add equalities:
-     | t1 = t1'
-     | ...
-     | t_{depth-1} = t'_{depth-1}
-     | t_{depth+1} = t'_{depth+1}
-     | ...
-     | t_{dim_sctr} = t'_{dim_sctr}
-
-     This means that all the time dimensions are equal except for
-     depth, where we will add t_{depth} = t'_{depth} + 1 in the next
-     step.  */
-  for (i = 0; i < dim_sctr; i++)
-    if (i != depth)
-      {
-        ppl_new_Linear_Expression_with_dimension (&le, new_dim);
-        ppl_set_coef (le, i, 1);
-        ppl_set_coef (le, i + offset, -1);
-        ppl_new_Constraint (&new_cstr, le, PPL_CONSTRAINT_TYPE_EQUAL);
-        ppl_Pointset_Powerset_C_Polyhedron_add_constraint (p2, new_cstr);
-        ppl_delete_Linear_Expression (le);
-        ppl_delete_Constraint (new_cstr);
-      }
-
-  /* Add equality : t_{depth} = t'_{depth} + 1.
-     This is the core part of this alogrithm, since this
-     constraint asks for the memory access stride (difference)
-     between two consecutive points in time dimensions.  */
-  {
-    ppl_new_Linear_Expression_with_dimension (&le, new_dim);
-    ppl_set_coef (le, depth, 1);
-    ppl_set_coef (le, depth + offset, -1);
-    ppl_set_inhomogeneous (le, 1);
-    ppl_new_Constraint (&new_cstr, le, PPL_CONSTRAINT_TYPE_EQUAL);
-    ppl_Pointset_Powerset_C_Polyhedron_add_constraint (p2, new_cstr);
-    ppl_delete_Linear_Expression (le);
-    ppl_delete_Constraint (new_cstr);
-  }
+  time_depth = psct_dynamic_dim (pbb, depth);
 
   /* P1 = P1 inter P2.  */
-  {
-    ppl_Pointset_Powerset_C_Polyhedron_intersection_assign (p1, p2);
-    ppl_delete_Pointset_Powerset_C_Polyhedron (p2);
-  }
+  ppl_Pointset_Powerset_C_Polyhedron_intersection_assign (p1, p2);
+  build_partial_difference (&p1, time_depth, offset, dim_sctr);
 
   /* Maximise the expression L2 - L1.  */
   {
@@ -262,12 +315,74 @@ memory_stride_in_loop (Value stride, graphite_dim_t depth, poly_dr_p pdr)
     ppl_set_coef (le, dim_L2, 1);
     ppl_set_coef (le, dim_L1, -1);
     ppl_max_for_le_pointset (p1, le, stride);
-    ppl_delete_Linear_Expression (le);
   }
+
+  if (dump_file && (dump_flags & TDF_DETAILS))
+    {
+      char *str;
+      void (*gmp_free) (void *, size_t);
+      
+      fprintf (dump_file, "\nStride in BB_%d, DR_%d, depth %d:",
+              pbb_index (pbb), PDR_ID (pdr), (int) depth);
+      str = mpz_get_str (0, 10, stride);
+      fprintf (dump_file, "  %s ", str);
+      mp_get_memory_functions (NULL, NULL, &gmp_free);
+      (*gmp_free) (str, strlen (str) + 1);
+    }
+
+  ppl_delete_Pointset_Powerset_C_Polyhedron (p1);
+  ppl_delete_Pointset_Powerset_C_Polyhedron (p2);
+  ppl_delete_Linear_Expression (le);
+}
+
+
+/* Sets STRIDES to the sum of all the strides of the data references
+   accessed in LOOP at DEPTH.  */
+
+static void
+memory_strides_in_loop_1 (lst_p loop, graphite_dim_t depth, mpz_t strides)
+{
+  int i, j;
+  lst_p l;
+  poly_dr_p pdr;
+  mpz_t s, n;
+
+  mpz_init (s);
+  mpz_init (n);
+
+  for (j = 0; VEC_iterate (lst_p, LST_SEQ (loop), j, l); j++)
+    if (LST_LOOP_P (l))
+      memory_strides_in_loop_1 (l, depth, strides);
+    else
+      for (i = 0; VEC_iterate (poly_dr_p, PBB_DRS (LST_PBB (l)), i, pdr); i++)
+       {
+         pdr_stride_in_loop (s, depth, pdr);
+         mpz_set_si (n, PDR_NB_REFS (pdr));
+         mpz_mul (s, s, n);
+         mpz_add (strides, strides, s);
+       }
+
+  mpz_clear (s);
+  mpz_clear (n);
 }
 
-/* Returns true when it is profitable to interchange time dimensions DEPTH1
-   and DEPTH2 with DEPTH1 < DEPTH2 for PBB.
+/* Sets STRIDES to the sum of all the strides of the data references
+   accessed in LOOP at DEPTH.  */
+
+static void
+memory_strides_in_loop (lst_p loop, graphite_dim_t depth, mpz_t strides)
+{
+  if (mpz_cmp_si (loop->memory_strides, -1) == 0)
+    {
+      mpz_set_si (strides, 0);
+      memory_strides_in_loop_1 (loop, depth, strides);
+    }
+  else
+    mpz_set (strides, loop->memory_strides);
+}
+
+/* Return true when the interchange of loops LOOP1 and LOOP2 is
+   profitable.
 
    Example:
 
@@ -307,7 +422,8 @@ memory_stride_in_loop (Value stride, graphite_dim_t depth, poly_dr_p pdr)
 
    Next, to measure the impact of iterating once in loop "i", we build
    a maximization problem: first, we add to DR accesses the dimensions
-   k, s2, s3, L1 = 100 * s0 + s1, L2, and D1: polyhedron P1.
+   k, s2, s3, L1 = 100 * s0 + s1, L2, and D1: this is the polyhedron P1.
+   L1 and L2 are the linearized memory access functions.
 
    | i   j   N   a  s0  s1   k  s2  s3  L1  L2  D1   1
    | 0   0   0   1   0   0   0   0   0   0   0   0  -5    = 0  alias = 5
@@ -347,42 +463,25 @@ memory_stride_in_loop (Value stride, graphite_dim_t depth, poly_dr_p pdr)
    profitable to interchange the loops at DEPTH1 and DEPTH2.  */
 
 static bool
-pbb_interchange_profitable_p (graphite_dim_t depth1, graphite_dim_t depth2,
-                             poly_bb_p pbb)
+lst_interchange_profitable_p (lst_p loop1, lst_p loop2)
 {
-  int i;
-  poly_dr_p pdr;
-  Value d1, d2, s, n;
+  mpz_t d1, d2;
   bool res;
 
-  gcc_assert (depth1 < depth2);
+  gcc_assert (loop1 && loop2
+             && LST_LOOP_P (loop1) && LST_LOOP_P (loop2)
+             && lst_depth (loop1) < lst_depth (loop2));
 
-  value_init (d1);
-  value_set_si (d1, 0);
-  value_init (d2);
-  value_set_si (d2, 0);
-  value_init (s);
-  value_init (n);
+  mpz_init (d1);
+  mpz_init (d2);
 
-  for (i = 0; VEC_iterate (poly_dr_p, PBB_DRS (pbb), i, pdr); i++)
-    {
-      value_set_si (n, PDR_NB_REFS (pdr));
-
-      memory_stride_in_loop (s, depth1, pdr);
-      value_multiply (s, s, n);
-      value_addto (d1, d1, s);
-
-      memory_stride_in_loop (s, depth2, pdr);
-      value_multiply (s, s, n);
-      value_addto (d2, d2, s);
-    }
+  memory_strides_in_loop (loop1, lst_depth (loop1), d1);
+  memory_strides_in_loop (loop2, lst_depth (loop2), d2);
 
   res = value_lt (d1, d2);
 
-  value_clear (d1);
-  value_clear (d2);
-  value_clear (s);
-  value_clear (n);
+  mpz_clear (d1);
+  mpz_clear (d2);
 
   return res;
 }
@@ -435,41 +534,68 @@ lst_apply_interchange (lst_p lst, int depth1, int depth2)
     pbb_interchange_loop_depths (depth1, depth2, LST_PBB (lst));
 }
 
-/* Return true when the interchange of loops at depths DEPTH1 and
-   DEPTH2 to all the statements below LST is profitable.  */
+/* Return true when the nest starting at LOOP1 and ending on LOOP2 is
+   perfect: i.e. there are no sequence of statements.  */
 
 static bool
-lst_interchange_profitable_p (lst_p lst, int depth1, int depth2)
+lst_perfectly_nested_p (lst_p loop1, lst_p loop2)
 {
-  if (!lst)
+  if (loop1 == loop2)
+    return true;
+
+  if (!LST_LOOP_P (loop1))
     return false;
 
-  if (LST_LOOP_P (lst))
-    {
-      int i;
-      lst_p l;
-      bool res = false;
+  return VEC_length (lst_p, LST_SEQ (loop1)) == 1
+    && lst_perfectly_nested_p (VEC_index (lst_p, LST_SEQ (loop1), 0), loop2);
+}
 
-      for (i = 0; VEC_iterate (lst_p, LST_SEQ (lst), i, l); i++)
-       {
-         bool profitable = lst_interchange_profitable_p (l, depth1, depth2);
+/* Transform the loop nest between LOOP1 and LOOP2 into a perfect
+   nest.  To continue the naming tradition, this function is called
+   after perfect_nestify.  NEST is set to the perfectly nested loop
+   that is created.  BEFORE/AFTER are set to the loops distributed
+   before/after the loop NEST.  */
+
+static void
+lst_perfect_nestify (lst_p loop1, lst_p loop2, lst_p *before,
+                    lst_p *nest, lst_p *after)
+{
+  poly_bb_p first, last;
 
-         if (profitable && !LST_LOOP_P (lst)
-             && dump_file && (dump_flags & TDF_DETAILS))
-           fprintf (dump_file,
-                    "Interchanging loops at depths %d and %d is profitable for stmt_%d.\n",
-                    depth1, depth2, pbb_index (LST_PBB (lst)));
+  gcc_assert (loop1 && loop2
+             && loop1 != loop2
+             && LST_LOOP_P (loop1) && LST_LOOP_P (loop2));
 
-         res |= profitable;
-       }
+  first = LST_PBB (lst_find_first_pbb (loop2));
+  last = LST_PBB (lst_find_last_pbb (loop2));
+
+  *before = copy_lst (loop1);
+  *nest = copy_lst (loop1);
+  *after = copy_lst (loop1);
+
+  lst_remove_all_before_including_pbb (*before, first, false);
+  lst_remove_all_before_including_pbb (*after, last, true);
+
+  lst_remove_all_before_excluding_pbb (*nest, first, true);
+  lst_remove_all_before_excluding_pbb (*nest, last, false);
 
-      return res;
+  if (lst_empty_p (*before))
+    {
+      free_lst (*before);
+      *before = NULL;
+    }
+  if (lst_empty_p (*after))
+    {
+      free_lst (*after);
+      *after = NULL;
+    }
+  if (lst_empty_p (*nest))
+    {
+      free_lst (*nest);
+      *nest = NULL;
     }
-  else
-    return pbb_interchange_profitable_p (depth1, depth2, LST_PBB (lst));
 }
 
-
 /* Try to interchange LOOP1 with LOOP2 for all the statements of the
    body of LOOP2.  LOOP1 contains LOOP2.  Return true if it did the
    interchange.  */
@@ -479,12 +605,28 @@ lst_try_interchange_loops (scop_p scop, lst_p loop1, lst_p loop2)
 {
   int depth1 = lst_depth (loop1);
   int depth2 = lst_depth (loop2);
+  lst_p transformed;
 
-  if (!lst_interchange_profitable_p (loop2, depth1, depth2))
+  lst_p before = NULL, nest = NULL, after = NULL;
+
+  if (!lst_interchange_profitable_p (loop1, loop2))
     return false;
 
+  if (!lst_perfectly_nested_p (loop1, loop2))
+    lst_perfect_nestify (loop1, loop2, &before, &nest, &after);
+
   lst_apply_interchange (loop2, depth1, depth2);
 
+  /* Sync the transformed LST information and the PBB scatterings
+     before using the scatterings in the data dependence analysis.  */
+  if (before || nest || after)
+    {
+      transformed = lst_substitute_3 (SCOP_TRANSFORMED_SCHEDULE (scop), loop1,
+                                     before, nest, after);
+      lst_update_scattering (transformed);
+      free_lst (transformed);
+    }
+
   if (graphite_legal_transform (scop))
     {
       if (dump_file && (dump_flags & TDF_DETAILS))
@@ -492,64 +634,86 @@ lst_try_interchange_loops (scop_p scop, lst_p loop1, lst_p loop2)
                 "Loops at depths %d and %d will be interchanged.\n",
                 depth1, depth2);
 
+      /* Transform the SCOP_TRANSFORMED_SCHEDULE of the SCOP.  */
+      lst_insert_in_sequence (before, loop1, true);
+      lst_insert_in_sequence (after, loop1, false);
+
+      if (nest)
+       {
+         lst_replace (loop1, nest);
+         free_lst (loop1);
+       }
+
       return true;
     }
 
   /* Undo the transform.  */
+  free_lst (before);
+  free_lst (nest);
+  free_lst (after);
   lst_apply_interchange (loop2, depth2, depth1);
   return false;
 }
 
-/* Try to interchange LOOP with all the loops contained in the body of
-   LST.  Return true if it did interchanged some loops.  */
+/* Selects the inner loop in LST_SEQ (INNER_FATHER) to be interchanged
+   with the loop OUTER in LST_SEQ (OUTER_FATHER).  */
 
 static bool
-lst_try_interchange (scop_p scop, lst_p loop, lst_p lst)
+lst_interchange_select_inner (scop_p scop, lst_p outer_father, int outer,
+                             lst_p inner_father)
 {
-  if (!lst)
-    return false;
+  int inner;
+  lst_p loop1, loop2;
 
-  if (LST_LOOP_P (lst))
-    {
-      int i;
-      lst_p l;
-      bool res = lst_try_interchange_loops (scop, loop, lst);
+  gcc_assert (outer_father
+             && LST_LOOP_P (outer_father)
+             && LST_LOOP_P (VEC_index (lst_p, LST_SEQ (outer_father), outer))
+             && inner_father
+             && LST_LOOP_P (inner_father));
 
-      for (i = 0; VEC_iterate (lst_p, LST_SEQ (lst), i, l); i++)
-       res |= lst_try_interchange (scop, loop, l);
+  loop1 = VEC_index (lst_p, LST_SEQ (outer_father), outer);
 
-      return res;
-    }
+  for (inner = 0; VEC_iterate (lst_p, LST_SEQ (inner_father), inner, loop2); inner++)
+    if (LST_LOOP_P (loop2)
+       && (lst_try_interchange_loops (scop, loop1, loop2)
+           || lst_interchange_select_inner (scop, outer_father, outer, loop2)))
+      return true;
 
   return false;
 }
 
-/* Interchanges all the loops of LST that are considered profitable to
-   interchange.  Return true if it did interchanged some loops.  */
+/* Interchanges all the loops of LOOP and the loops of its body that
+   are considered profitable to interchange.  Return true if it did
+   interchanged some loops.  OUTER is the index in LST_SEQ (LOOP) that
+   points to the next outer loop to be considered for interchange.  */
 
 static bool
-lst_do_interchange (scop_p scop, lst_p lst)
+lst_interchange_select_outer (scop_p scop, lst_p loop, int outer)
 {
-  if (!lst)
+  lst_p l;
+  bool res = false;
+  int i = 0;
+  lst_p father;
+
+  if (!loop || !LST_LOOP_P (loop))
     return false;
 
-  if (LST_LOOP_P (lst))
+  father = LST_LOOP_FATHER (loop);
+  if (father)
     {
-      int i;
-      lst_p l;
-      bool res = false;
-
-      if (lst_depth (lst) >= 0)
-       for (i = 0; VEC_iterate (lst_p, LST_SEQ (lst), i, l); i++)
-         res |= lst_try_interchange (scop, lst, l);
-
-      for (i = 0; VEC_iterate (lst_p, LST_SEQ (lst), i, l); i++)
-       res |= lst_do_interchange (scop, l);
-
-      return res;
+      while (lst_interchange_select_inner (scop, father, outer, loop))
+       {
+         res = true;
+         loop = VEC_index (lst_p, LST_SEQ (father), outer);
+       }
     }
 
-  return false;
+  if (LST_LOOP_P (loop))
+    for (i = 0; VEC_iterate (lst_p, LST_SEQ (loop), i, l); i++)
+      if (LST_LOOP_P (l))
+       res |= lst_interchange_select_outer (scop, l, i);
+
+  return res;
 }
 
 /* Interchanges all the loop depths that are considered profitable for SCOP.  */
@@ -557,22 +721,12 @@ lst_do_interchange (scop_p scop, lst_p lst)
 bool
 scop_do_interchange (scop_p scop)
 {
-  bool transform_done = false;
-
-  store_scattering (scop);
+  bool res = lst_interchange_select_outer
+    (scop, SCOP_TRANSFORMED_SCHEDULE (scop), 0);
 
-  transform_done = lst_do_interchange (scop, SCOP_TRANSFORMED_SCHEDULE (scop));
-
-  if (!transform_done)
-    return false;
+  lst_update_scattering (SCOP_TRANSFORMED_SCHEDULE (scop));
 
-  if (!graphite_legal_transform (scop))
-    {
-      restore_scattering (scop);
-      return false;
-    }
-
-  return transform_done;
+  return res;
 }