espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-22 19:09:31 +08:00
Code Issues Packages Projects Releases Wiki Activity

gdb/arm: Don't rely on loop detection to stop unwinding

Browse Source 
Setting SP of the next frame to the same address as the current frame
is an ugly way to stop the unwinding.  A cleaner way is to rely on
the frame_unwind_stop_reason function to return UNWIND_OUTERMOST.

Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
This commit is contained in:
Torbjörn SVENSSON
2022-10-14 15:34:24 +02:00
parent 488054071b
commit 619cce4cac
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
gdb/arm-tdep.c
View File

@ -3356,7 +3356,6 @@ arm_m_exception_cache (frame_info_ptr this_frame)
to the exception and if FPU is used (causing extended stack frame). */ to the exception and if FPU is used (causing extended stack frame). */
CORE_ADDR lr = get_frame_register_unsigned (this_frame, ARM_LR_REGNUM); CORE_ADDR lr = get_frame_register_unsigned (this_frame, ARM_LR_REGNUM);
CORE_ADDR sp = get_frame_register_unsigned (this_frame, ARM_SP_REGNUM);
/* ARMv7-M Architecture Reference "A2.3.1 Arm core registers" /* ARMv7-M Architecture Reference "A2.3.1 Arm core registers"
states that LR is set to 0xffffffff on reset. ARMv8-M Architecture states that LR is set to 0xffffffff on reset. ARMv8-M Architecture
@ -3364,8 +3363,8 @@ arm_m_exception_cache (frame_info_ptr this_frame)
reset if Main Extension is implemented, otherwise the value is unknown. */ reset if Main Extension is implemented, otherwise the value is unknown. */
if (lr == 0xffffffff) if (lr == 0xffffffff)
{ {
/* Terminate any further stack unwinding by referring to self. */ /* Terminate any further stack unwinding. */
arm_cache_set_active_sp_value (cache, tdep, sp); arm_cache_set_active_sp_value (cache, tdep, 0);
return cache; return cache;
} }
@ -3387,8 +3386,8 @@ arm_m_exception_cache (frame_info_ptr this_frame)
{ {
warning (_("Non-secure to secure stack unwinding disabled.")); warning (_("Non-secure to secure stack unwinding disabled."));
/* Terminate any further stack unwinding by referring to self. */ /* Terminate any further stack unwinding. */
arm_cache_set_active_sp_value (cache, tdep, sp); arm_cache_set_active_sp_value (cache, tdep, 0);
return cache; return cache;
} }
@ -3452,8 +3451,8 @@ arm_m_exception_cache (frame_info_ptr this_frame)
{ {
warning (_("Non-secure to secure stack unwinding disabled.")); warning (_("Non-secure to secure stack unwinding disabled."));
/* Terminate any further stack unwinding by referring to self. */ /* Terminate any further stack unwinding. */
arm_cache_set_active_sp_value (cache, tdep, sp); arm_cache_set_active_sp_value (cache, tdep, 0);
return cache; return cache;
} }