On Tue, Jul 30, 2019 at 9:27 AM Martin Liška <mli...@suse.cz> wrote:
>
> Hi.
>
> We have to clean up dangling pointers before we call ggc_free for a 
> cgraph_edge.
>
> Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
> And it survives --enable-checking=release bootstrap on x86_64-linux-gnu.
>
> Ready to be installed?

Eh?  The only "real" effect I see is that e->indirect_info test is now
never true.

I think it rather means the edge we ggc_free is still referenced to
from somewhere
and _that_ needs to be fixed or we ggc_free the edge wrongly.

Richard.

> Thanks,
> Martin
>
> gcc/ChangeLog:
>
> 2019-07-30  Martin Liska  <mli...@suse.cz>
>
>         PR ipa/89330
>         * cgraph.c (symbol_table::free_edge): Memset 0 to cgraph_edge
>         before we call ggc_free.
> ---
>  gcc/cgraph.c | 2 ++
>  1 file changed, 2 insertions(+)
>
>

Reply via email to