TeamHeptaMirrors/gcc
1
0
Fork 1
mirror of https://github.com/gcc-mirror/gcc.git synced 2026-05-06 14:59:39 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix memory leak in vect_analyze_slp

Browse Source 
When I made the function to fail when SLP discovery fails I failed
to release the BST map.

	* tree-vect-slp.cc (vect_analyze_slp): Release BST map when
	returning.
This commit is contained in:
Richard Biener
2026-03-17 14:42:43 +01:00
committed by Richard Biener
parent cc3370077c
commit c16aebb8b4
1 changed files with 25 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
gcc/tree-vect-slp.cc
View File

@@ -5739,7 +5739,10 @@ vect_analyze_slp (vec_info *vinfo, unsigned max_tree_size,
slp_inst_kind_store, max_tree_size, &limit,
force_single_lane)
&& loop_vinfo)
return opt_result::failure_at (vect_location, "SLP build failed.\n");
{
release_scalar_stmts_to_slp_tree_map (bst_map);
return opt_result::failure_at (vect_location, "SLP build failed.\n");
}
/* For loops also start SLP discovery from non-grouped stores. */
if (loop_vinfo)
@@ -5760,8 +5763,11 @@ vect_analyze_slp (vec_info *vinfo, unsigned max_tree_size,
if (! vect_build_slp_instance (vinfo, slp_inst_kind_store,
stmts, roots, remain, max_tree_size,
&limit, bst_map, force_single_lane))
return opt_result::failure_at (vect_location,
"SLP build failed.\n");
{
release_scalar_stmts_to_slp_tree_map (bst_map);
return opt_result::failure_at (vect_location,
"SLP build failed.\n");
}
}
stmt_vec_info stmt_info;
@@ -5775,8 +5781,11 @@ vect_analyze_slp (vec_info *vinfo, unsigned max_tree_size,
if (! vect_build_slp_instance (vinfo, slp_inst_kind_store,
stmts, roots, remain, max_tree_size,
&limit, bst_map, force_single_lane))
return opt_result::failure_at (vect_location,
"SLP build failed.\n");
{
release_scalar_stmts_to_slp_tree_map (bst_map);
return opt_result::failure_at (vect_location,
"SLP build failed.\n");
}
}
}
@@ -5807,7 +5816,10 @@ vect_analyze_slp (vec_info *vinfo, unsigned max_tree_size,
/* Find SLP sequences starting from groups of reductions. */
if (!vect_analyze_slp_reductions (loop_vinfo, max_tree_size, &limit,
bst_map, force_single_lane))
return opt_result::failure_at (vect_location, "SLP build failed.\n");
{
release_scalar_stmts_to_slp_tree_map (bst_map);
return opt_result::failure_at (vect_location, "SLP build failed.\n");
}
/* Make sure to vectorize only-live stmts, usually inductions. */
for (edge e : get_loop_exit_edges (LOOP_VINFO_LOOP (loop_vinfo)))
@@ -5836,8 +5848,11 @@ vect_analyze_slp (vec_info *vinfo, unsigned max_tree_size,
stmts, roots, remain,
max_tree_size, &limit,
bst_map, force_single_lane))
return opt_result::failure_at (vect_location,
"SLP build failed.\n");
{
release_scalar_stmts_to_slp_tree_map (bst_map);
return opt_result::failure_at (vect_location,
"SLP build failed.\n");
}
}
}
@@ -5860,6 +5875,7 @@ vect_analyze_slp (vec_info *vinfo, unsigned max_tree_size,
|| !integer_zerop (args1))
{
roots.release ();
release_scalar_stmts_to_slp_tree_map (bst_map);
return opt_result::failure_at (vect_location,
"SLP build failed.\n");
}
@@ -5881,6 +5897,7 @@ vect_analyze_slp (vec_info *vinfo, unsigned max_tree_size,
bst_map, force_single_lane))
{
roots.release ();
release_scalar_stmts_to_slp_tree_map (bst_map);
return opt_result::failure_at (vect_location,
"SLP build failed.\n");
}