On Mon, 17 May 2021 14:12:00 +0200
Bernd Edlinger <bernd.edlin...@hotmail.de> wrote:
> > */ @@ -8715,19 +8770,26 @@ static tree
> > build_struct_group (struct gimplify_omp_ctx *ctx,
> > enum omp_region_type region_type, enum
> > tree_code code, tree decl, unsigned int *flags, tree c,
> > - hash_map<tree, tree> *&struct_map_to_clause,
> > + hash_map<tree_operand_hash, tree>
> > *&struct_map_to_clause, tree *&prev_list_p, tree *&list_p, bool
> > *cont) {
> > poly_offset_int coffset;
> > poly_int64 cbitpos;
> > - tree base_ref;
> > + tree base_ind, base_ref;
> > + tree *list_in_p = list_p, *prev_list_in_p = prev_list_p;
> >
>
> Is this a kind of debug code?
> This fails to compile:
>
> ../../gcc-trunk/gcc/gimplify.c: In function ‘tree_node*
> build_struct_group(gimplify_omp_ctx*, omp_region_type, tree_code,
> tree, unsigned int*, tree, hash_map<tree_operand_hash, tree_node*>*&,
> tree_node**&, tree_node**&, bool*)’:
> ../../gcc-trunk/gcc/gimplify.c:8779:9: error: unused variable
> ‘list_in_p’ [-Werror=unused-variable] 8779 | tree *list_in_p =
> list_p, *prev_list_in_p = prev_list_p; | ^~~~~~~~~
> ../../gcc-trunk/gcc/gimplify.c:8779:30: error: unused variable
> ‘prev_list_in_p’ [-Werror=unused-variable] 8779 | tree *list_in_p =
> list_p, *prev_list_in_p = prev_list_p; |
> ^~~~~~~~~~~~~~
Oops, that's left over from an earlier iteration of the patch, and
indeed isn't needed any more. I'll be sure to bootstrap the next
iteration of these patches I send upstream.
Thanks,
Julian
