aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTakayuki 'January June' Suwa <jjsuwa_sys3175@yahoo.co.jp>2023-02-18 13:54:10 +0900
committerMax Filippov <jcmvbkbc@gmail.com>2023-02-20 17:32:31 -0800
commita804419c89db9e1c0b99e772dda6c95095bab0bf (patch)
treec0e46cf88e38167dcc9ab0611050e756a5c80b6d
parent65c64105a816a887092b954dee0340e45c9495b4 (diff)
downloadgcc-a804419c89db9e1c0b99e772dda6c95095bab0bf.zip
gcc-a804419c89db9e1c0b99e772dda6c95095bab0bf.tar.gz
gcc-a804419c89db9e1c0b99e772dda6c95095bab0bf.tar.bz2
xtensa: Enforce return address saving when -Og is specified
Leaf function often omits saving its return address to the stack slot, and this feature often makes debugging very confusing, especially for stack dump analysis. gcc/ChangeLog: * config/xtensa/xtensa.cc (xtensa_call_save_reg): Change to return true if register A0 (return address register) when -Og is specified.
-rw-r--r--gcc/config/xtensa/xtensa.cc7
1 files changed, 5 insertions, 2 deletions
diff --git a/gcc/config/xtensa/xtensa.cc b/gcc/config/xtensa/xtensa.cc
index 3e2e22d..d0320ef 100644
--- a/gcc/config/xtensa/xtensa.cc
+++ b/gcc/config/xtensa/xtensa.cc
@@ -3223,8 +3223,11 @@ xtensa_call_save_reg (int regno)
return false;
if (regno == A0_REG)
- return crtl->profile || !crtl->is_leaf || crtl->calls_eh_return ||
- df_regs_ever_live_p (regno);
+ /* Ensure the return address to be saved to the stack slot in order
+ to assist stack dump analysis when -Og is specified. */
+ return optimize_debug
+ || crtl->profile || !crtl->is_leaf || crtl->calls_eh_return
+ || df_regs_ever_live_p (regno);
if (crtl->calls_eh_return && IN_RANGE (regno, 2, 3))
return true;