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

--- Comment #3 from rguenther at suse dot de <rguenther at suse dot de> ---
On Tue, 14 Jan 2025, jakub at gcc dot gnu.org wrote:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116068
> 
> Jakub Jelinek <jakub at gcc dot gnu.org> changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>                  CC|                            |jakub at gcc dot gnu.org
> 
> --- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> So shall we go for something like:
> --- gcc/cgraphunit.cc   2025-01-02 11:23:03.119703892 +0100
> +++ gcc/cgraphunit.cc   2025-01-14 12:23:41.918872660 +0100
> @@ -311,6 +311,7 @@ symbol_table::process_new_functions (voi
>      {
>        cgraph_node *node = cgraph_new_nodes[i];
>        fndecl = node->decl;
> +      bitmap_obstack_initialize (NULL);
>        switch (state)
>         {
>         case CONSTRUCTION:
> @@ -367,6 +368,7 @@ symbol_table::process_new_functions (voi
>           gcc_unreachable ();
>           break;
>         }
> +      bitmap_obstack_release (NULL);
>      }
> 
>    cgraph_new_nodes.release ();
> 
> ?  All the other spots that execute some pass list in cgraphunit.cc wrap that
> with bitmap_obstack_initialize/release.

That looks correct to me.

Reply via email to