[PATCH] Fix lost locus issues at -O0 -g (PRs debug/29609, debug/36690, debug/37616)

Ian Lance Taylor iant@google.com
Tue Oct 7 18:41:00 GMT 2008


Jakub Jelinek <jakub@redhat.com> writes:

> I was afraid next_nonnote_insn could fall through over a BB note into
> a different BB.  Guess I could skip over other notes by hand, or
> just iterate through insns, until either INSN_P is found or BB_END (e->dest)
> is passed.

OK, fair enough.


>> > --- gcc/testsuite/gcc.dg/debug/pr37616.c.jj	2008-10-01 14:35:14.000000000 +0200
>> > +++ gcc/testsuite/gcc.dg/debug/pr37616.c	2008-10-01 15:50:15.000000000 +0200
>> > @@ -0,0 +1,37 @@
>> > +/* PR debug/37616 */
>> > +/* Test that one can put breakpoints onto continue, exitlab and break
>> > +   and actually see program reaching those breakpoints.  */
>> > +/* { dg-do run } */
>> > +/* { dg-options "-O0 -g -dA" } */
>> 
>> I don't understand how this test tests what the comment describes.
>
> The debug/pr*.c tests obviously only test whether you can compile/link/run the
> program, the debug/dwarf2/pr*.c test are almost identical and test whether
> the desired locus is emitted and the intent is that the same tests will be
> used in gdb testsuite where it can be all tested more thoroughly.

Please add appropriate comments to the tests.

OK in any case.

Thanks.

Ian



More information about the Gcc-patches mailing list