From: "Pierre Muller" <muller@ics.u-strasbg.fr>
To: "'Pedro Alves'" <pedro_alves@portugalmail.pt>,
"'gdb-patches'" <gdb-patches@sourceware.org>
Subject: RE: skip __main
Date: Tue, 29 Jan 2008 08:50:00 -0000 [thread overview]
Message-ID: <000f01c8624b$c284ee20$478eca60$@u-strasbg.fr> (raw)
In-Reply-To: <479EB897.5030603@portugalmail.pt>
Hi Pedro!
Thanks for resubmitting this.
The problem for me is that I am currently
unable to do a testsuite on cygwin,
the test stays forever on gdb.threads/manythreads.exp
test and the timeout does not seem to work there :(
Concerning your patch:
I would suggest one change:
instead of using hard coded "main" name,
you should use the main_name function from symtab.c source
defined in symtab.h
I am not really sure this is useful,
(the modified main_name for GNU pascal compiler
does not contain a call to __main), but
it seems more logical to me.
Pierre
> -----Original Message-----
> From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> owner@sourceware.org] On Behalf Of Pedro Alves
> Sent: Tuesday, January 29, 2008 6:25 AM
> To: Pierre Muller; gdb-patches
> Subject: skip __main
>
> Hi all, Pierre,
>
> Here's the much smaller patch I mentioned at gdb@.
>
> This version is about as small as it can get. First the prologue is
> analysed and skipped, then line info is used to get at the first line
> of user code in the function. If the function is called "main" (humm,
> I shall move that into the callback, as I believe the place where
> __main/__gccmain is emitted is configurable on the gcc side), call a
> gdbarch method that skips the __main call. Works OK as long as nothing
> is scheduled into before the __main call. That means that it should
> work OK at -O0, which is enough for fixing the runto_main problems in
> the testsuite. Then again, debugging code with .stabs debug info at
> anything but -O0 is not pleasant anyway.
>
> Last time I looked at the results, it uncovered other more serious
> problems, so it was a win just for that fact.
>
> I'm starting an overnight testrun on Cygwin, as the last time I did
> that was a couple of months back.
>
> Any comments? Suggestions for a better gdbarch callback name?
>
> --
> Pedro Alves
next prev parent reply other threads:[~2008-01-29 7:51 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-29 5:35 Pedro Alves
2008-01-29 8:50 ` Pierre Muller [this message]
2008-01-29 15:48 ` Pedro Alves
2008-01-29 16:20 ` Pierre Muller
2008-01-29 20:26 ` Pedro Alves
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='000f01c8624b$c284ee20$478eca60$@u-strasbg.fr' \
--to=muller@ics.u-strasbg.fr \
--cc=gdb-patches@sourceware.org \
--cc=pedro_alves@portugalmail.pt \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox