Message ID | 20211215155447.19379-18-frederik@codesourcery.com |
---|---|
State | New |
Headers | show |
Series | OpenACC "kernels" Improvements | expand |
Another comment-only change. Otherwise, just re-diffed Frederik's patch. Mostly s/.c/.cc/, but I added one '. ' that got lost. On 15.12.21 16:54, Frederik Harwath wrote: > * graphite-sese-to-poly.c (build_poly_sr_1): Fix a typo and > a reference to a variable which does not exist. > * graphite-isl-ast-to-gimple.c (gsi_insert_earliest): Fix typo > in comment. OK for mainline? Tobias ----------------- 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
On Mon, 16 May 2022, Tobias Burnus wrote: > Another comment-only change. > > Otherwise, just re-diffed Frederik's patch. Mostly s/.c/.cc/, but I > added one '. ' that got lost. > > On 15.12.21 16:54, Frederik Harwath wrote: > > * graphite-sese-to-poly.c (build_poly_sr_1): Fix a typo and > > a reference to a variable which does not exist. > > * graphite-isl-ast-to-gimple.c (gsi_insert_earliest): Fix typo > > in comment. > > OK for mainline? OK. Richard. > Tobias > ----------------- > 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 --git a/gcc/graphite-isl-ast-to-gimple.c b/gcc/graphite-isl-ast-to-gimple.c index 1ad68a1d4735..0712d85b67a6 100644 --- a/gcc/graphite-isl-ast-to-gimple.c +++ b/gcc/graphite-isl-ast-to-gimple.c @@ -1018,7 +1018,7 @@ gsi_insert_earliest (gimple_seq seq) basic_block begin_bb = get_entry_bb (codegen_region); /* Inserting the gimple statements in a vector because gimple_seq behave - in strage ways when inserting the stmts from it into different basic + in strange ways when inserting the stmts from it into different basic blocks one at a time. */ auto_vec<gimple *, 3> stmts; for (gimple_stmt_iterator gsi = gsi_start (seq); !gsi_end_p (gsi); diff --git a/gcc/graphite-sese-to-poly.c b/gcc/graphite-sese-to-poly.c index 204d382ed4cc..33d6a98327b8 100644 --- a/gcc/graphite-sese-to-poly.c +++ b/gcc/graphite-sese-to-poly.c @@ -649,14 +649,14 @@ build_poly_sr_1 (poly_bb_p pbb, gimple *stmt, tree var, enum poly_dr_type kind, isl_map *acc, isl_set *subscript_sizes) { scop_p scop = PBB_SCOP (pbb); - /* Each scalar variables has a unique alias set number starting from + /* Each scalar variable has a unique alias set number starting from the maximum alias set assigned to a dr. */ int alias_set = scop->max_alias_set + SSA_NAME_VERSION (var); subscript_sizes = isl_set_fix_si (subscript_sizes, isl_dim_set, 0, alias_set); /* Add a constrain to the ACCESSES polyhedron for the alias set of - data reference DR. */ + the reference */ isl_constraint *c = isl_equality_alloc (isl_local_space_from_space (isl_map_get_space (acc))); c = isl_constraint_set_constant_si (c, -alias_set);