On Sat, 26 Mar 2022, Jakub Jelinek wrote:
> Hi!
>
> The recent change didn't initialize comp_step while previously we used
> XCNEW to allocate it.
>
> I think RS_ANY is better than RS_INTERNAL (== 0) as the default.
>
> Bootstrappedd/regtested on x86_64-linux and i686-linux, ok for trunk?
OK.
> 2022-03-26 Jakub Jelinek <ja...@redhat.com>
>
> PR tree-optimization/105056
> * tree-predcom.cc (component::component): Initialize also comp_step.
>
> --- gcc/tree-predcom.cc.jj 2022-03-24 12:23:11.000000000 +0100
> +++ gcc/tree-predcom.cc 2022-03-25 17:50:09.238282600 +0100
> @@ -367,7 +367,8 @@ enum ref_step_type
>
> struct component
> {
> - component (bool es) : eliminate_store_p (es), next (NULL) {}
> + component (bool es) : comp_step (RS_ANY), eliminate_store_p (es),
> + next (NULL) {}
>
> /* The references in the component. */
> auto_vec<dref> refs;
>
> Jakub
>
>
--
Richard Biener <rguent...@suse.de>
SUSE Software Solutions Germany GmbH, Maxfeldstrasse 5, 90409 Nuernberg,
Germany; GF: Ivo Totev; HRB 36809 (AG Nuernberg)
