diff mbox series

[OG11,committed,04/22] graphite: Rename isl_id_for_ssa_name

Message ID	20211117160330.20029-5-frederik@codesourcery.com
State	Committed
Headers	DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 404003858412 IronPort-SDR: LeYFzHUO8NSM6hEqzzrXcOFWtEx3QnVL15uUZP6oRVXjp/BXucYWdbrf3KBVvKTT7xEuVRCCJQ Ye12eu7JqzWuQudwZRyQzce2i6KvaVcHVaUV/A3Piadri7l8p2FYEltJIeq62qONqFXeRKsAIu vjwqXQ85qd/CcT9GDLktSvMc36mjC8uuhNZvNseTdVC/WRQP/BfIuW1Ayzw/2ywiSDhcFx5A1o tpqxBWC8JUCckl8WjdX1ZXPhF+UCzLX7Ze98wd0cnLiDd8TlZUR3LRhepxEd+8i4h6CpJjgbRa tdoX9BXPswdh0nhmRkFmAIXo IronPort-SDR: ZzVjkaBJcOKRlkVEo9bA7BeZzWerH5g+YFSZaBJXAN1wUoht4lNji9c6NqxbJoc7Y9ijWPi9Ts dqAatGHcBZYOBV8+HRAMHQ+RGc/9r6xUORxQmFgCx5c1i+YdXOiHg4iGgEKX7Eo7pymt5dkhwW WPbHYUo82pBwYfkgRj7nH2SDwW/Mpx+OQnCri7CHhG0hXAPXBvm3irx4Xpf2GcY1ZiA02fmsUD z/ClzC4UsGmkM61RPLCY97GtGX6LJt9MRln+Gn+B0gP0n7PW5YJh+sc0sIA5nwhDTVCtodpPSY Fwc= From: Frederik Harwath <frederik@codesourcery.com> To: <gcc-patches@gcc.gnu.org> Subject: [OG11][committed][PATCH 04/22] graphite: Rename isl_id_for_ssa_name Date: Wed, 17 Nov 2021 17:03:13 +0100 Message-ID: <20211117160330.20029-5-frederik@codesourcery.com> In-Reply-To: <20211117160330.20029-1-frederik@codesourcery.com> References: <20211117160330.20029-1-frederik@codesourcery.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain Precedence: list Errors-To: gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org Sender: "Gcc-patches" <gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org>
Series	OpenACC "kernels" Improvements \| [OG11,committed,00/22] OpenACC "kernels" Improvements [OG11,committed,01/22] Fortran: delinearize multi-dimensional array accesses [OG11,committed,02/22] openacc: Move pass_oacc_device_lower after pass_graphite [OG11,committed,03/22] graphite: Extend SCoP detection dump output [OG11,committed,04/22] graphite: Rename isl_id_for_ssa_name [OG11,committed,05/22] graphite: Fix minor mistakes in comments [OG11,committed,07/22] Move compute_alias_check_pairs to tree-data-ref.c [OG11,committed,08/22] graphite: Add runtime alias checking [OG11,committed,09/22] openacc: Use Graphite for dependence analysis in "kernels" regions [OG11,committed,10/22] openacc: Add "can_be_parallel" flag info to "graph" dumps [OG11,committed,11/22] openacc: Add further kernels tests [OG11,committed,12/22] openacc: Remove unused partitioning in "kernels" regions [OG11,committed,13/22] Add function for printing a single OMP_CLAUSE [OG11,committed,14/22] openacc: Add data optimization pass [OG11,committed,15/22] openacc: Add runtime alias checking for OpenACC kernels [OG11,committed,16/22] openacc: Warn about "independent" "kernels" loops with data-dependences [OG11,committed,17/22] openacc: Handle internal function calls in pass_lim [OG11,committed,18/22] openacc: Disable pass_pre on outlined functions analyzed by Graphite [OG11,committed,19/22] graphite: Tune parameters for OpenACC use [OG11,committed,20/22] graphite: Adjust scop loop-nest choice [OG11,committed,21/22] graphite: Accept loops without data references [OG11,committed,22/22] openacc: Adjust test expectations to new "kernels" handling

Commit Message

Frederik Harwath Nov. 17, 2021, 4:03 p.m. UTC

  The SSA names for which this function gets used are always SCoP
parameters and hence "isl_id_for_parameter" is a better name.  It also
explains the prefix "P_" for those names in the ISL representation.

gcc/ChangeLog:

        * graphite-sese-to-poly.c (isl_id_for_ssa_name): Rename to ...
          (isl_id_for_parameter): ... this new function name.
          (build_scop_context): Adjust function use.
---
 gcc/graphite-sese-to-poly.c | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

--
2.33.0

-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

diff mbox series

Patch

diff --git a/gcc/graphite-sese-to-poly.c b/gcc/graphite-sese-to-poly.c
index eebf2e02cfca..195851cb540a 100644
--- a/gcc/graphite-sese-to-poly.c
+++ b/gcc/graphite-sese-to-poly.c
@@ -100,14 +100,15 @@  extract_affine_mul (scop_p s, tree e, __isl_take isl_space *space)
   return isl_pw_aff_mul (lhs, rhs);
 }

-/* Return an isl identifier from the name of the ssa_name E.  */
+/* Return an isl identifier for the parameter P.  */

 static isl_id *
-isl_id_for_ssa_name (scop_p s, tree e)
+isl_id_for_parameter (scop_p s, tree p)
 {
-  char name1[14];
-  snprintf (name1, sizeof (name1), "P_%d", SSA_NAME_VERSION (e));
-  return isl_id_alloc (s->isl_context, name1, e);
+  gcc_checking_assert (TREE_CODE (p) == SSA_NAME);
+  char name[14];
+  snprintf (name, sizeof (name), "P_%d", SSA_NAME_VERSION (p));
+  return isl_id_alloc (s->isl_context, name, p);
 }

 /* Return an isl identifier for the data reference DR.  Data references and
@@ -893,15 +894,15 @@  build_scop_context (scop_p scop)
   isl_space *space = isl_space_set_alloc (scop->isl_context, nbp, 0);

   unsigned i;
-  tree e;
-  FOR_EACH_VEC_ELT (region->params, i, e)
+  tree p;
+  FOR_EACH_VEC_ELT (region->params, i, p)
     space = isl_space_set_dim_id (space, isl_dim_param, i,
-                                  isl_id_for_ssa_name (scop, e));
+                                  isl_id_for_parameter (scop, p));

   scop->param_context = isl_set_universe (space);

-  FOR_EACH_VEC_ELT (region->params, i, e)
-    add_param_constraints (scop, i, e);
+  FOR_EACH_VEC_ELT (region->params, i, p)
+    add_param_constraints (scop, i, p);
 }

 /* Return true when loop A is nested in loop B.  */