OpenE2K
/
gcc
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix memory leak in ipa-pure-const 

* ipa-pure-const.c (set_function_state): Remove an existing
	funct_state.
	(remove_node_data): Do not free it as it's released
	in set_function_state.

From-SVN: r236469
This commit is contained in:
Martin Liska 2016-05-19 17:06:17 +02:00 committed by Martin Liska
parent 6e078af89e
commit 7f153d8202
2 changed files with 15 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gcc/ChangeLog
View File

@ -1,3 +1,10 @@
2016-05-19 Martin Liska <mliska@suse.cz>
* ipa-pure-const.c (set_function_state): Remove an existing
funct_state.
(remove_node_data): Do not free it as it's released
in set_function_state.
2016-05-19 Martin Liska <mliska@suse.cz>
* tree-vect-slp.c (vect_attempt_slp_rearrange_stmts): Release

14
gcc/ipa-pure-const.c
View File

@ -258,6 +258,13 @@ set_function_state (struct cgraph_node *node, funct_state s)
if (!funct_state_vec.exists ()
|| funct_state_vec.length () <= (unsigned int)node->uid)
funct_state_vec.safe_grow_cleared (node->uid + 1);
/* If funct_state_vec already contains a funct_state, we have to release
it before it's going to be ovewritten. */
if (funct_state_vec[node->uid] != NULL
&& funct_state_vec[node->uid] != &varying_state)
free (funct_state_vec[node->uid]);
funct_state_vec[node->uid] = s;
}
@ -956,12 +963,7 @@ static void
remove_node_data (struct cgraph_node *node, void *data ATTRIBUTE_UNUSED)
{
if (has_function_state (node))
{
funct_state l = get_function_state (node);
if (l != &varying_state)
free (l);
set_function_state (node, NULL);
}
set_function_state (node, NULL);
}