[PATCH v3] Add dll trampoline code handling for windows 64bit
Roland Schwingel
roland@onevision.com
Fri Mar 16 09:56:00 GMT 2012
Hi,
I hope this is the last round on this...
> Kai Tietz read this patch -- he says "hello" and asked me to respond.
>
> Roland> + struct minimal_symbol *indsym =
> Roland> + indirect ? lookup_minimal_symbol_by_pc (pc +
indirect) : 0;
>
> Kai says that PC-relative expressions are relative to the end of the
> instruction. So, 'pc + indirect' is incorrect here and also in the
> read_memory call.
Oh man! I couldn't see the forest for the trees.
While my previous approach was in the end correctly working this is
the cleaner solution. That way read_memory_unsigned_integer()
can be used now, delivering correct results.
ChangeLog:
2012-03-16 Roland Schwingel <roland.schwingel@onevision.com>
* amd64-windows-tdep.c: #include "frame.h".
(amd64_windows_skip_trampoline_code): New function.
(amd64_windows_init_abi): Add trampoline registration.
Roland
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: amd64-windows-tdep.c.patch
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20120316/21da370e/attachment.ksh>
More information about the Gdb-patches
mailing list