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 <[email protected]> > > 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 <[email protected]> SUSE Software Solutions Germany GmbH, Maxfeldstrasse 5, 90409 Nuernberg, Germany; GF: Ivo Totev; HRB 36809 (AG Nuernberg)
