Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@mvista.com>
To: law@redhat.com
Cc: gdb-patches@sources.redhat.com
Subject: Re: Test for GCC debug symbol bug
Date: Fri, 04 Jan 2002 17:55:00 -0000	[thread overview]
Message-ID: <20020104205623.A4968@nevyn.them.org> (raw)
In-Reply-To: <1754.1010182499@porcupine.cygnus.com>

On Fri, Jan 04, 2002 at 03:14:59PM -0700, law@redhat.com wrote:
> 
> This patch adds a test for a GCC bug in its line number output.
> 
> Basically if we have a multi-line IF or WHILE conditional, GCC will
> emit incorrect line numbers for the IF/WHILE statement.
> 
> The incorrect line numbers (of course) make debugging multi-line 
> IF/WHILE statements a PITA.
> 
> To help ensure the GCC team doesn't break this again, I'd like to 
> add this relatively simple test to the GDB testsuite.
> 
> 
> 	* gdb.base/break.c (multi_line_if_conditional): New function.
> 	(multi_ilne_while_conditional): Likewise.
> 	* gdb.base/break.exp: Verify that a breakpoint on a multi-line
> 	IF or WHILE condition puts the breakpoint at the start of
> 	the condition.

<Andrew Cagney Hat On, apologies to Andrew>

This is obvious; please do commit it.  I'm all in favor of debug info
tests.

Of course, we may want to mark it XFAIL if we end up stopping where
broken GCC's would have; I really dislike FAILs that GDB can do nothing
about, although in some cases they're unavoidable.  Do we have a
documented policy for what all the DejaGNU result codes mean in the GDB
testsuite?  If not, should I propose one?

-- 
Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer


      reply	other threads:[~2002-01-05  1:55 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-01-04 14:17 law
2002-01-04 17:55 ` Daniel Jacobowitz [this message]

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=20020104205623.A4968@nevyn.them.org \
    --to=drow@mvista.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=law@redhat.com \
    /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