From 109c8d3934b3748a2fdb061c9a250b30b54fdf5b Mon Sep 17 00:00:00 2001 From: Jan Hubicka Date: Wed, 17 Jan 2007 00:42:06 +0100 Subject: [PATCH] tree-ssanames.c (release_dead_ssa_names): Instead of ggc_freeing the names... * tree-ssanames.c (release_dead_ssa_names): Instead of ggc_freeing the names, just unlink the chain so we don't crash on dangling pointers to dead SSA names. From-SVN: r120837 --- gcc/ChangeLog | 6 ++++++ gcc/tree-ssanames.c | 7 ++++++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 7d44816fc9e..a8e0c18d834 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2007-01-16 Jan Hubicka + + * tree-ssanames.c (release_dead_ssa_names): Instead of ggc_freeing + the names, just unlink the chain so we don't crash on dangling pointers + to dead SSA names. + 2007-01-16 Jan Hubicka * cgraph.h (cgraph_decide_inlining_incrementally): Kill. diff --git a/gcc/tree-ssanames.c b/gcc/tree-ssanames.c index d9ab940fa90..07b83f831af 100644 --- a/gcc/tree-ssanames.c +++ b/gcc/tree-ssanames.c @@ -325,7 +325,12 @@ release_dead_ssa_names (void) for (t = FREE_SSANAMES (cfun); t; t = next) { next = TREE_CHAIN (t); - ggc_free (t); + /* Dangling pointers might make GGC to still see dead SSA names, so it is + important to unlink the list and avoid GGC from seeing all subsequent + SSA names. In longer run we want to have all dangling pointers here + removed (since they usually go trhough dead statements that consume + considerable amounts of memory). */ + TREE_CHAIN (t) = NULL_TREE; n++; } FREE_SSANAMES (cfun) = NULL; -- 2.30.2