Re: [PATCH] Fix up msymbol type of dll trampoline to mst_solib_trampoline

[Eli Zaretskii <eliz@gnu.org>]

> Date: Sat, 6 Jul 2013 15:20:35 +0800
> From: Yao Qi <yao@codesourcery.com>
> CC: <asmwarrior@gmail.com>, <tromey@redhat.com>, <gdb-patches@sourceware.org>
> 
> On 07/05/2013 10:29 PM, Eli Zaretskii wrote:
> >> Date: Fri, 5 Jul 2013 20:22:18 +0800
> >> From: Yao Qi <yao@codesourcery.com>
> >> CC: Tom Tromey <tromey@redhat.com>, <gdb-patches@sourceware.org>
> >>
> >>> BTW:
> >>> 1, It looks like I can't set a breakpoint on a function with is __stdcall calling convention
> >>> When I type:
> >>> (gdb) b Add_S
> >>> Function "Add_S" not defined.
> >>> In fact, the symbol name about "Add_S" function is "Add_S@8", (This can be seen from the disassembler)
> >>> I don't know how to set such breakpoint by function names.
> >>
> >> I am not an expert on windows, but it is a bug to me.
> >
> > What exactly do you see as a bug here?
> >
> 
> We can't set breakpoint on 'Add_S' in current GDB,
> 
> (gdb) b Add_S
> Function "Add_S" not defined.
> Make breakpoint pending on future shared library load? (y or [n])
> 
> IMO, It is expected that 'b Add_S' can set a breakpoint on Add_S.

Then GDB should look for Add_S@n symbols, where n is the number of
bytes in the function's arguments.