https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109231

--- Comment #28 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot 
Uni-Bielefeld.DE> ---
> --- Comment #25 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> Anyway, as I said for the second version, it would be nice to also try
> subvariants:
> //  relayout_decl (DECL_RESULT (new_fndecl));
>   for (tree parm = DECL_ARGUMENTS (new_fndecl); parm; parm = DECL_CHAIN 
> (parm))
>     relayout_decl (parm);
>   targetm.target_option.relayout_function (new_fndecl);
> //  aggregate_value_p (DECL_RESULT (new_fndecl), new_fndecl);
> and
>   relayout_decl (DECL_RESULT (new_fndecl));
> //  for (tree parm = DECL_ARGUMENTS (new_fndecl); parm; parm = DECL_CHAIN
> (parm))
> //    relayout_decl (parm);
>   targetm.target_option.relayout_function (new_fndecl);
> //  aggregate_value_p (DECL_RESULT (new_fndecl), new_fndecl);
> and
> //  relayout_decl (DECL_RESULT (new_fndecl));
> //  for (tree parm = DECL_ARGUMENTS (new_fndecl); parm; parm = DECL_CHAIN
> (parm))
> //    relayout_decl (parm);
>   targetm.target_option.relayout_function (new_fndecl);
>   aggregate_value_p (DECL_RESULT (new_fndecl), new_fndecl);
> so see if the comparison failure is fixed by the result relayout, or by
> argument
> relayout or by the aggregate_value_p call actually having some side-effects
> other than return value.

Of those three subvariants, only the third one passed the build without
the comparison failure.

Reply via email to