Re: [PATCH] [gdb/testsuite] Add gdb.base/include-main.exp

[Tom Tromey <tom@tromey.com>]

>>>>> "Tom" == Tom de Vries via Gdb-patches <gdb-patches@sourceware.org> writes:

Tom> Add a C test-case that mimics gdb.ada/dgopt.exp, that is:
Tom> - generate debug info as described above,
Tom> - issue a list of a line in include-main.c, while the corresponding
Tom>   CU is not expanded yet.

I noted this in another patch submission, but I thought I'd point it out
here: this test case also fails for me, with git gdb, with the fission
and fission-dwp target boards.  I'm using the Fedora 34 system gcc.

E.g.:

    $ runtest --target_board=fission gdb.base/include-main.exp
    [...]
    Running /home/tromey/gdb/build/gdb/testsuite/../../../binutils-gdb/gdb/testsuite/gdb.base/include-main.exp ...
    FAIL: gdb.base/include-main.exp: list include-main.c:18

Given this, I wonder if we really want to continue supporting this.  I
think it may be better to just declare these as compiler bugs (including
gdb.ada/dgopt.exp) when using fission.

The rationale for this is just that the line table and the CU DIE ought
to agree, and if they don't, GDB can just declare that it respects the
line table and may ignore the CU DIE.

What do you think of this?

Fixing this in the fission case is a pain because the GDB index is
intentionally set up to avoid reading the CU DIE unless the CU is going
to be expanded.

thanks,
Tom