[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