This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, 3/6] Add recursion to make_param_constraints
- From: Richard Biener <rguenther at suse dot de>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: Richard Biener <richard dot guenther at gmail dot com>, "gcc-patches at gnu dot org" <gcc-patches at gnu dot org>
- Date: Mon, 2 Nov 2015 16:25:14 +0100 (CET)
- Subject: Re: [PATCH, 3/6] Add recursion to make_param_constraints
- Authentication-results: sourceware.org; auth=none
- References: <562E0CF5 dot 8000606 at mentor dot com> <CAFiYyc20gSLqPSyF9-_xYjN7-RPB2xqsfuM4j45XNU4fv-R8Eg at mail dot gmail dot com> <562E5381 dot 5 at mentor dot com> <562F26E2 dot 40906 at mentor dot com> <562F6D1A dot 4010001 at mentor dot com> <562F748D dot 5020507 at mentor dot com> <563653F5 dot 9090302 at mentor dot com> <5636561E dot 2030200 at mentor dot com>
On Sun, 1 Nov 2015, Tom de Vries wrote:
> On 01/11/15 19:03, Tom de Vries wrote:
> >
> > So, the new patch series is:
> >
> > 1 Rename make_restrict_var_constraints to make_param_constraints
> > 2 Handle recursive restrict in function parameter
> >
> > I'll repost in reply to this message.
>
> This no-functional-changes patch:
> - moves the one constraint handling loop left in
> intra_create_variable_infos to make_restrict_var_constraints
> - renames make_restrict_var_constraints to make_param_constraints
> - adds a parameter toplevel to make_param_constraints to distinguish
> between the two calling contexts
> - adds a parmeter restrict_name that allows to pass in the name of
> restrict vars
>
> This patch was posted before at
> https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03111.html .
+ if (toplevel)
+ make_constraint_from (vi, nonlocal_id);
+ else
+ make_copy_constraint (vi, nonlocal_id);
I think make_constraint_from is what we want in both cases.
Ok with this change (thus drop the toplevel parameter).
Richard.