https://gcc.gnu.org/g:953bf37690d22de956d75c6aef7a9690ad55b9a7
commit r13-8923-g953bf37690d22de956d75c6aef7a9690ad55b9a7
Author: LIU Hao <lh_mo...@126.com>
Date: Mon Jul 15 16:55:52 2024 +0800
Do not use caller-saved registers for COMDAT functions
A reference to a COMDAT function may be resolved to another definition
outside the current translation unit, so it's not eligible for `-fipa-ra`.
In `decl_binds_to_current_def_p()` there is already a check for weak
symbols. This commit checks for COMDAT functions that are not implemented
as weak symbols, for example, on *-*-mingw32.
gcc/ChangeLog:
PR rtl-optimization/115049
* varasm.cc (decl_binds_to_current_def_p): Add a check for COMDAT
declarations too, like weak ones.
(cherry picked from commit 5080840d8fbf25a321dd27543a1462d393d338bc)
Diff:
---
gcc/varasm.cc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/gcc/varasm.cc b/gcc/varasm.cc
index 9ad41c3dbdb8..e95cef8a7259 100644
--- a/gcc/varasm.cc
+++ b/gcc/varasm.cc
@@ -7632,6 +7632,8 @@ decl_binds_to_current_def_p (const_tree decl)
for all other declaration types. */
if (DECL_WEAK (decl))
return false;
+ if (DECL_COMDAT_GROUP (decl))
+ return false;
if (DECL_COMMON (decl)
&& (DECL_INITIAL (decl) == NULL
|| (!in_lto_p && DECL_INITIAL (decl) == error_mark_node)))
