Re: [PATCH] libgcc: Expose the current instruction pointer in SEH _Unwind_Backtrace

2020-09-04 Thread Martin Storsjö

On Tue, 1 Sep 2020, Martin Storsjö wrote:


Previously, the SEH version of _Unwind_Backtrace did unwind
the stack and call the provided callback function as intended,
but there was little the caller could do within the callback to
actually get any info about that particular level in the unwind.

Set the ra pointer, which is used by _Unwind_GetIP, to allow
using this function to inspect the state within the callback,
before calling the callback function.

2020-09-01  Martin Storsjö  

libgcc/Changelog:
   * unwind-seh.c (_Unwind_Backtrace): Set the ra pointer before
   calling the callback.
---
libgcc/unwind-seh.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/libgcc/unwind-seh.c b/libgcc/unwind-seh.c
index 1a70180cfaa..72473135862 100644
--- a/libgcc/unwind-seh.c
+++ b/libgcc/unwind-seh.c
@@ -466,6 +466,8 @@ _Unwind_Backtrace(_Unwind_Trace_Fn trace,
_context.disp->HandlerData,
_context.disp->EstablisherFrame, NULL);

+  gcc_context.ra = ms_context.Rip;
+
  /* Call trace function.  */
  if (trace (_context, trace_argument) != _URC_NO_REASON)
return _URC_FATAL_PHASE1_ERROR;


Ping - any comments on this one?

// Martin


[PATCH] libgcc: Expose the current instruction pointer in SEH _Unwind_Backtrace

2020-09-01 Thread Martin Storsjö
Previously, the SEH version of _Unwind_Backtrace did unwind
the stack and call the provided callback function as intended,
but there was little the caller could do within the callback to
actually get any info about that particular level in the unwind.

Set the ra pointer, which is used by _Unwind_GetIP, to allow
using this function to inspect the state within the callback,
before calling the callback function.

2020-09-01  Martin Storsjö  

libgcc/Changelog:
* unwind-seh.c (_Unwind_Backtrace): Set the ra pointer before
calling the callback.
---
 libgcc/unwind-seh.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libgcc/unwind-seh.c b/libgcc/unwind-seh.c
index 1a70180cfaa..72473135862 100644
--- a/libgcc/unwind-seh.c
+++ b/libgcc/unwind-seh.c
@@ -466,6 +466,8 @@ _Unwind_Backtrace(_Unwind_Trace_Fn trace,
_context.disp->HandlerData,
_context.disp->EstablisherFrame, NULL);
 
+  gcc_context.ra = ms_context.Rip;
+
   /* Call trace function.  */
   if (trace (_context, trace_argument) != _URC_NO_REASON)
return _URC_FATAL_PHASE1_ERROR;
-- 
2.17.1