From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2205) id C03213858CDA; Tue, 4 Oct 2022 14:51:07 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C03213858CDA DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1664895067; bh=zwRjiUBVZPAqOmmyZiO+Yzvj465tXbh3dM7/kxwaWy4=; h=From:To:Subject:Date:From; b=u1YePAjj3ZbZ+Euku4mXi1rZR9ywCA+PkyHGyV/2KaSzdcqdhRzKs3hf1EUlrqSd7 IANtQMBRp9EjER7N6C4XE0q1jOE8MztSUZ7XNVja3UbQ+nXzp4jQXTLOSTPHymb7N4 pAi2sseDjxvdMFQbZIciWbcosdFbwzm5YJ1kEjos= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Tom de Vries To: gdb-cvs@sourceware.org Subject: [binutils-gdb] [gdb/symtab] Don't complain about inlined functions X-Act-Checkin: binutils-gdb X-Git-Author: Tom de Vries X-Git-Refname: refs/heads/master X-Git-Oldrev: 436a7b5ef27e6d866a631d6020e904321cbee7e8 X-Git-Newrev: 3aeba5cd1c7a2eac0d8a0efd051beaa77522f614 Message-Id: <20221004145107.C03213858CDA@sourceware.org> Date: Tue, 4 Oct 2022 14:51:07 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3D3aeba5cd1c7a= 2eac0d8a0efd051beaa77522f614 commit 3aeba5cd1c7a2eac0d8a0efd051beaa77522f614 Author: Tom de Vries Date: Tue Oct 4 16:51:03 2022 +0200 [gdb/symtab] Don't complain about inlined functions =20 With the test-case included in this patch, we get: ... (gdb) ptype main^M During symbol reading: cannot get low and high bounds for subprogram DI= E \ at 0x113^M During symbol reading: cannot get low and high bounds for subprogram DI= E \ at 0x11f^M type =3D int (void)^M (gdb) FAIL: gdb.dwarf2/inline.exp: ptype main ... =20 The complaints are about foo, with DW_AT_inline =3D=3D DW_INL_inlined: ... <1><11f>: Abbrev Number: 6 (DW_TAG_subprogram) <120> DW_AT_name : foo <126> DW_AT_prototyped : 1 <126> DW_AT_type : <0x10c> <12a> DW_AT_inline : 1 (inlined) ... and foo2, with DW_AT_inline =3D=3D DW_INL_declared_inlined: ... <1><113>: Abbrev Number: 5 (DW_TAG_subprogram) <114> DW_AT_name : foo2 <11a> DW_AT_prototyped : 1 <11a> DW_AT_type : <0x10c> <11e> DW_AT_inline : 3 (declared as inline and inlined) ... =20 Fix this by not complaining about inlined functions. =20 Tested on x86_64-linux. Diff: --- gdb/dwarf2/read.c | 9 ++++++++- gdb/testsuite/gdb.dwarf2/inline.c | 34 ++++++++++++++++++++++++++++++++++ gdb/testsuite/gdb.dwarf2/inline.exp | 32 ++++++++++++++++++++++++++++++++ 3 files changed, 74 insertions(+), 1 deletion(-) diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c index 35cce9396ae..812d9c608ef 100644 --- a/gdb/dwarf2/read.c +++ b/gdb/dwarf2/read.c @@ -12044,7 +12044,14 @@ read_func_scope (struct die_info *die, struct dwar= f2_cu *cu) <=3D PC_BOUNDS_INVALID) { attr =3D dwarf2_attr (die, DW_AT_external, cu); - if (attr =3D=3D nullptr || !attr->as_boolean ()) + bool external_p =3D attr !=3D nullptr && attr->as_boolean (); + attr =3D dwarf2_attr (die, DW_AT_inline, cu); + bool inlined_p + =3D (attr !=3D nullptr + && attr->is_nonnegative () + && (attr->as_nonnegative () =3D=3D DW_INL_inlined + || attr->as_nonnegative () =3D=3D DW_INL_declared_inlined)); + if (!external_p && !inlined_p) complaint (_("cannot get low and high bounds " "for subprogram DIE at %s"), sect_offset_str (die->sect_off)); diff --git a/gdb/testsuite/gdb.dwarf2/inline.c b/gdb/testsuite/gdb.dwarf2/i= nline.c new file mode 100644 index 00000000000..2e07f6c4a9d --- /dev/null +++ b/gdb/testsuite/gdb.dwarf2/inline.c @@ -0,0 +1,34 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2022 Free Software Foundation, Inc. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . = */ + +static int __attribute__((always_inline)) +foo (void) +{ + return 0; +} + +static int inline __attribute__((always_inline)) +foo2 (void) +{ + return 0; +} + +int +main (void) +{ + return foo () + foo2 (); +} diff --git a/gdb/testsuite/gdb.dwarf2/inline.exp b/gdb/testsuite/gdb.dwarf2= /inline.exp new file mode 100644 index 00000000000..06182d877af --- /dev/null +++ b/gdb/testsuite/gdb.dwarf2/inline.exp @@ -0,0 +1,32 @@ +# Copyright 2022 Free Software Foundation, Inc. + +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +standard_testfile .c + +if [prepare_for_testing "failed to prepare" $testfile $srcfile \ + {debug nowarnings}] { + return -1 +} + +set cmd "ptype main" + +set re \ + [multi_line \ + $cmd \ + "type =3D int \\(void\\)"] + +with_complaints 5 { + gdb_test $cmd $re "$cmd without complaints" +}