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

avoid-store-forwarding: Fix sbitmap memory leak in process_store_forwarding

Browse Source 
forwarded_bytes was allocated with sbitmap_alloc but never freed on any
of the function's return paths.  Use auto_sbitmap instead, which calls
sbitmap_free in its destructor.

gcc/ChangeLog:

	* avoid-store-forwarding.cc
	(store_forwarding_analyzer::process_store_forwarding): Use
	auto_sbitmap for forwarded_bytes to ensure it is freed on all
	return paths.

Reviewed-By: Konstantinos Eleftheriou <konstantinos.eleftheriou@vrull.eu>
This commit is contained in:
Philipp Tomsich
2026-02-23 11:05:06 +01:00
parent 34fb8d5131
commit 429a5fe400
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gcc/avoid-store-forwarding.cc
View File

@@ -176,9 +176,7 @@ process_store_forwarding (vec<store_fwd_info> &stores, rtx_insn *load_insn,
We can also eliminate stores on addresses that are overwritten
by later stores. */
sbitmap forwarded_bytes = sbitmap_alloc (load_size);
bitmap_clear (forwarded_bytes);
auto_sbitmap forwarded_bytes (load_size);
unsigned int i;
store_fwd_info* it;
auto_vec<store_fwd_info> redundant_stores;