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

[PATCH] ipa-reorder-for-locality - Fix PR123749 avoid creating new partition if no more nodes left to process

Browse Source 
This patch fixes PR123749 which caused by an empty partition created in WPA
partitioning pass as a result of -fipa-reorder-for-locality creating one.
This also fixes an indentation issue.

Bootstrapped and regtested on aarch64-none-linux-gnu. OK for mainline?

Signed-off-by: Prachi Godbole <pgodbole@nvidia.com>

gcc/Changelog:

	PR target/123749
	* ipa-locality-cloning.cc (partition_callchain): Start new partition
	if no. of insns exceeds size and if there are nodes to process.
This commit is contained in:
Prachi Godbole
2026-03-17 22:24:00 -07:00
parent 801c6fbeec
commit 0f8e6cf83b
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
gcc/ipa-locality-cloning.cc
View File

@@ -1187,8 +1187,6 @@ partition_callchain (cgraph_node *node, locality_partition &partition,
/* Aliases are added in the same partition as their targets.
Aliases are not cloned and their callees are not processed separately. */
cgraph_node *cl_node = NULL;
if (partition->insns > partition_size)
partition = create_partition (npartitions);
/* Iterate over all unique callees of NODE, direct callees and callees via
inlined nodes. This avoids calling partition_callchain () separately for
@@ -1205,10 +1203,13 @@ partition_callchain (cgraph_node *node, locality_partition &partition,
{
if (!node_partitioned_p (n))
{
if (partition->insns > partition_size)
partition = create_partition (npartitions);
add_node_to_partition (partition, n);
if (dump_file)
fprintf (dump_file, "Partitioned node: %s\n",
n->dump_asm_name ());
fprintf (dump_file, "Partitioned node: %s\n",
n->dump_asm_name ());
partition_callchain (n, partition, cloning_model, freq_cutoff,
size, cl_num, npartitions, partition_size,
scheme);