On Mon, 2016-05-02 at 11:50 -0600, Martin Sebor wrote:
> Hi Jan,
>
> I just noticed the compilation errors in the attached file with
> the latest trunk. It seems as though your recent patch below may
> be incomplete:
>
> commit 46e5dccc6f188bd0fd5af4e9778f547ab63c9cae
> Author: hubicka <hubicka@138bc75d-0d04-0410-961f-82ee72b054a4>
> Date: Mon May 2 16:55:56 2016 +0000
>
> The following change causes compilation errors due to
> ipa_find_agg_cst_for_param taking just three arguments, while it
> is being called with four. (I haven't looked into the other error.)
>
> Regards
> Martin
>
> --- a/gcc/ipa-inline-analysis.c
> +++ b/gcc/ipa-inline-analysis.c
> @@ -850,7 +850,8 @@ evaluate_conditions_for_known_args (struct
> cgraph_node *node
> ,
> if (known_aggs.exists ())
> {
> agg = known_aggs[c->operand_num];
> - val = ipa_find_agg_cst_for_param (agg, c->offset, c
> ->by_ref);
> + val = ipa_find_agg_cst_for_param (agg,
> known_vals[c->operand_num],
> + c->offset, c
> ->by_ref);
I saw this too (with r235766). I believe it's fixed by r235770 and
r235771:
2016-05-02 Jan Hubicka <[email protected]>
* cif-code.def (CIF_THUNK): Add.
* ipa-inline-analsysis.c (evaluate_conditions_for_known_args): Revert
accidental change.
(albeit with a typo in that second filename)
r235771 work for me, FWIW.
Dave
