gdb/testsuite: skip gdb.opt/inline-entry.exp for gcc 7 and older

It was pointed out that the recently added gdb.opt/inline-entry.exp
test would fail when run using gcc 7 and earlier, on an x86-64 target:

  https://inbox.sourceware.org/gdb-patches/9fe35ea1-d99b-444d-bd1b-e3a1f108dd77@suse.de

Bernd Edlinger points out that, for gcc, the test relies on the
-gstatement-frontiers work which was added in gcc 8.x:

  https://inbox.sourceware.org/gdb-patches/DU2PR08MB10263357597688D9D66EA745CE4242@DU2PR08MB10263.eurprd08.prod.outlook.com

For gcc 7.x and older, without the -gstatement-frontiers work, the
compiler uses DW_AT_entry_pc differently, which leads to a poorer
debug experience.

Here is the interesting source line from inline-entry.c:

  if ((global && bar (1)) || bar (2))

And here's some of the relevant disassembly output:

  Dump of assembler code for function main:
     0x401020 <+0>:	mov    0x3006(%rip),%eax	(1)
     0x401026 <+6>:	test   %eax,%eax		(2)
     0x401028 <+8>:	mov    0x2ffe(%rip),%eax	(3)
     0x40102e <+14>:	je     0x401038 <main+24>	(4)
     0x401030 <+16>:	sub    $0x1,%eax		(5)
     0x401033 <+19>:	jne    0x40103d <main+29>	(6)

Lines (1), (2), and (4) represent the check of 'global'.  However,
line (3) is actually the first instruction for 'bar' which has been
inlined.  Lines (5) and (6) are also part of the first inlined 'bar'
function.

If the check of 'global' returns false then the first call to 'bar'
should never happen, this is accomplished by the branch at (4) being
taken.

For gcc 8+, gcc generates a DW_AT_entry_pc with the value 0x401030,
this is where GDB places a breakpoint for 'bar', and this address is
after the branch at line (4), and so, if the call to 'bar' never
happens, the breakpoint is never hit.

For gcc 7 and older, gcc generates a DW_AT_entry_pc with the value
0x401028, which is the first address associated with the inline 'bar'
function.  Unfortunately, this address is also before the check of
'global' has completed, this means that GDB hits the 'bar' breakpoint
before the inferior has decided if 'bar' should actually be called or
not.

I don't think there's really much GDB can do in the older gcc
versions, we are placing the breakpoint at the entry point, and this
is within bar.  Given that this test does really depend on the newer
gcc behaviour, I think the only sensible solution is to skip this test
when an older version of gcc is being used.

I've incorporated the check for -gstatement-frontiers support that
Bernd suggested and now the test will be skipped for older versions of
GCC.

Approved-By: Tom de Vries <tdevries@suse.de>

0 files changed, 0 insertions, 0 deletions