RE: [EXTERNAL] Re: [PATCH][PUSHED] Fix cfg maintenance after inlining in AutoFDO

2023-05-09 Thread Eugene Rozenfeld via Gcc-patches
>> free_dominance_info (CDI_DOMINATORS); @@ -1674,7 +1677,7 @@ >> auto_profile (void) >> pop_cfun (); >>} >> >> - return TODO_rebuild_cgraph_edges; >> + return 0; >This change isn't mentioned - was it accidential? No, it wasn't accidental. There is no reason to return TODO_rebui

Re: [EXTERNAL] Re: [PATCH][PUSHED] Fix cfg maintenance after inlining in AutoFDO

2023-05-09 Thread Richard Biener via Gcc-patches
On Wed, May 10, 2023 at 12:05 AM Eugene Rozenfeld wrote: > > >> free_dominance_info (CDI_DOMINATORS); @@ -1674,7 +1677,7 @@ > >> auto_profile (void) > >> pop_cfun (); > >>} > >> > >> - return TODO_rebuild_cgraph_edges; > >> + return 0; > > >This change isn't mentioned - was it acci