From patchwork Thu Oct 31 19:19:22 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bernd Edlinger X-Patchwork-Id: 35518 Received: (qmail 117858 invoked by alias); 31 Oct 2019 19:19:28 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 117849 invoked by uid 89); 31 Oct 2019 19:19:28 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-16.8 required=5.0 tests=AWL, BAYES_00, FREEMAIL_FROM, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, KAM_STOCKGEN, RCVD_IN_DNSWL_NONE, SPF_HELO_PASS, SPF_PASS autolearn=ham version=3.3.1 spammy= X-HELO: EUR03-DB5-obe.outbound.protection.outlook.com Received: from mail-oln040092071096.outbound.protection.outlook.com (HELO EUR03-DB5-obe.outbound.protection.outlook.com) (40.92.71.96) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 31 Oct 2019 19:19:25 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=G8AN403xH9n0NMPhkR+2wGIibKqyq10efIv3AG+1q/5NlJuutT63HO4peW8KD6hTUqlfPBHbcW+UjmuZcShOSTu/T6Rng9IyP/ApWdC0lvmF0Ip6yelZkDE5ay+S+YqS0k7A3yj3rarl/PuRQpd8p8DNAwNu7oqa84Sfkr6/UQyuGEB4siIWaf+BzxBHCFsIkOgh3tv8PW4ZBtbxaDHpwgdlm1aKYd54gwqsJbl41W871imyYOTsjsbUihU3jbr2YFSmspHy0PExl7C6Rfp2PCCRjLFKTg+3hRZ/sfRbOa9xdZSKebauhpmKv4k7KIlzKaqJfcRwkX9v1ZuUAlIMJg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=apyQEoXsfrhqlDij9PH7aB++fZS9IXy3x5uhLGRoJ+g=; b=PMLHRMZZHreVdynpj2h7tjfSigvpfwoJzGg4WCd6/lgkqhbIWo6TDeOnY9R3bOx62gHN6UpKWjQbeLtoMxQ48Ey6OnZnRbY2VAx76wzHFjRaWJ9W0NcIqxE/a7ksR/z73Mvt1u/8P9TPCtULCpATMaF88SCnkfT/N7iifU4RxfZaB5bmsIRqaGgkqO6BjAB0HJpD8+A3LfILTVmp7oG5t0lN7cF2KZJx2UNxf7l0LsoGXZSV1O04bOdzqg4lKeQHHVC2tVk7IOl6NAh+1Fi9B4lhn38vtdCOP0N1FOGQ49mUTulWj8jAnlx7pAUr91hthpRrJWUm34eMXmJrsIQy8A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none Received: from AM5EUR03FT029.eop-EUR03.prod.protection.outlook.com (10.152.16.54) by AM5EUR03HT101.eop-EUR03.prod.protection.outlook.com (10.152.17.146) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.2387.20; Thu, 31 Oct 2019 19:19:22 +0000 Received: from AM6PR03MB4519.eurprd03.prod.outlook.com (10.152.16.57) by AM5EUR03FT029.mail.protection.outlook.com (10.152.16.150) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2387.20 via Frontend Transport; Thu, 31 Oct 2019 19:19:22 +0000 Received: from AM6PR03MB4519.eurprd03.prod.outlook.com ([fe80::a9e4:70ae:47f8:70f1]) by AM6PR03MB4519.eurprd03.prod.outlook.com ([fe80::a9e4:70ae:47f8:70f1%5]) with mapi id 15.20.2387.028; Thu, 31 Oct 2019 19:19:22 +0000 From: Bernd Edlinger To: Pedro Alves , Simon Marchi , "gdb-patches@sourceware.org" Subject: [PATCHv3] Make "skip" work on inline frames Date: Thu, 31 Oct 2019 19:19:22 +0000 Message-ID: References: <8fc93db4-8906-4f4e-53f4-225ebfa0115d@simark.ca> <215bbf9c-4c3c-5cd2-c657-51aa7262f234@simark.ca> In-Reply-To: x-microsoft-original-message-id: <9e010caa-37cf-996f-6ddc-d51c3092967b@hotmail.de> x-ms-exchange-purlcount: 2 x-ms-exchange-transport-forked: True MIME-Version: 1.0 On 10/31/19 7:00 PM, Pedro Alves wrote: > On 10/31/19 4:53 PM, Simon Marchi wrote: >> On 2019-10-31 12:42 p.m., Pedro Alves wrote: >>> On 10/30/19 9:56 PM, Bernd Edlinger wrote: >>>> +if { [prepare_for_testing "failed to prepare" "skip2" \ >>>> + {skip2.c skip1.c } \ >>>> + {debug nowarnings optimize=-O2}] } { >>> >>> Instead of -O2, could you make this use -O0 (the default), >>> and then use attribute((always_inline)) to force inlining? >>> We do that in some tests. E.g., gdb.opt/inline-locals.c. >> >> I think that's a good suggestion, but just be aware that there used to be >> some problems with always_inline, e.g.: > > I don't think always_inline changes anything in the debug info special, > it just tells the compiler to inline the function even at -O0, which is > what we're after. > >> >> https://sourceware.org/bugzilla/show_bug.cgi?id=13263 > > This one seems to be about attribute((artificial)), and the desire > to not step into such functions automatically: > > "Given that I marked the function always_inline and artificial, I > was expecting not to step into its body." > >> https://sourceware.org/bugzilla/show_bug.cgi?id=12429 >> > > This one seems like a generic inlining issue. > >> I'm not sure if those are still valid. If they are, it might be more difficult >> that expected to use always_inline. > I don't think always_inline is anything special compared to inlining > because of -O2. > > Thanks, > Pedro Alves > Ah, thanks for that hint! always_inline works quite well. The debug session started (using gcc 4.8.4) with -O2 -g on the line with "{" in main, and with -O0 -g at the first real statement, so I had to remove one step. I did not worry about it initially, but now I agree that would have caused trouble. I attached both parts of the patch, the fist part unchanged from previous. The test case now makes sure to not repeat the names. Thanks Bernd. From d8fc12de59c47c94ef42f91e0793554df1e4d714 Mon Sep 17 00:00:00 2001 From: Bernd Edlinger Date: Wed, 30 Oct 2019 21:35:22 +0100 Subject: [PATCH 2/2] Add a test case for skip with inlined functions --- gdb/testsuite/gdb.base/skip2.c | 64 ++++++++++++++++++++++++++++++++ gdb/testsuite/gdb.base/skip2.exp | 80 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 144 insertions(+) create mode 100644 gdb/testsuite/gdb.base/skip2.c create mode 100644 gdb/testsuite/gdb.base/skip2.exp diff --git a/gdb/testsuite/gdb.base/skip2.c b/gdb/testsuite/gdb.base/skip2.c new file mode 100644 index 0000000..d1abd45 --- /dev/null +++ b/gdb/testsuite/gdb.base/skip2.c @@ -0,0 +1,64 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2019 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 . */ + +#include + +int bar (void); +int baz (int); +void skip1_test_skip_file_and_function (void); +void test_skip_file_and_function (void); + +__attribute__((__always_inline__)) static inline int +foo (void) +{ + return bar (); +} + +int +main () +{ + volatile int x; + + /* step immediately into the inlined code */ + x = baz (foo ()); + + /* step first over non-inline code, this involves a different code path */ + x = 0; x = baz (foo ()); + + test_skip_file_and_function (); + + return 0; +} + +static void +test_skip (void) +{ +} + +static void +end_test_skip_file_and_function (void) +{ + abort (); +} + +void +test_skip_file_and_function (void) +{ + test_skip (); + skip1_test_skip_file_and_function (); + end_test_skip_file_and_function (); +} diff --git a/gdb/testsuite/gdb.base/skip2.exp b/gdb/testsuite/gdb.base/skip2.exp new file mode 100644 index 0000000..a59dd0f --- /dev/null +++ b/gdb/testsuite/gdb.base/skip2.exp @@ -0,0 +1,80 @@ +# Copyright 2019 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 . + +load_lib completion-support.exp + +standard_testfile + +if { [prepare_for_testing "failed to prepare" "skip2" \ + {skip2.c skip1.c } \ + {debug nowarnings}] } { + return -1 +} + +set srcfile skip2.c +set srcfile1 skip1.c + +if ![runto_main] { + fail "can't run to main" + return +} + +# Create a skiplist entry for a specified file and function. + +gdb_test "skip function foo" "Function foo will be skipped when stepping\." + +gdb_test "bt" "\\s*\\#0\\s+main.*" "in the main" +gdb_test "step" ".*" "step into baz, since foo will be skipped" +gdb_test "bt" "\\s*\\#0\\s+baz.*" "in the baz, since foo was skipped" +gdb_test "step" ".*" "step in the baz" +gdb_test "bt" "\\s*\\#0\\s+baz.*" "still in the baz" +gdb_test "step" ".*" "step back to main" +gdb_test "bt" "\\s*\\#0\\s+main.*" "again in the main" +gdb_test "step" ".*" "step again into baz, since foo will be skipped" +gdb_test "bt" "\\s*\\#0\\s+baz.*" "again in the baz" +gdb_test "step" ".*" "step in the baz, again" +gdb_test "bt" "\\s*\\#0\\s+baz.*" "still in the baz, again" +gdb_test "step" ".*" "step back to main, again" +gdb_test "bt" "\\s*\\#0\\s+main.*" "again back to main" + +if ![runto_main] { + fail "can't run to main" + return +} + +with_test_prefix "double step" { + gdb_test "bt" "\\s*\\#0\\s+main.*" "in the main" + gdb_test "step 2" ".*" "step into baz, since foo will be skipped" + gdb_test "bt" "\\s*\\#0\\s+baz.*" "still in the baz" + gdb_test "step" ".*" "step back to main" + gdb_test "bt" "\\s*\\#0\\s+main.*" "again in the main" + gdb_test "step 2" ".*" "step again into baz, since foo will be skipped" + gdb_test "bt" "\\s*\\#0\\s+baz.*" "again in the baz" + gdb_test "step" ".*" "step back to main, again" + gdb_test "bt" "\\s*\\#0\\s+main.*" "again back to main" +} + +if ![runto_main] { + fail "can't run to main" + return +} + +with_test_prefix "triple step" { + gdb_test "bt" "\\s*\\#0\\s+main.*" "in the main" + gdb_test "step 3" ".*" "step over baz" + gdb_test "bt" "\\s*\\#0\\s+main.*" "again in the main" + gdb_test "step 3" ".*" "step over baz, again" + gdb_test "bt" "\\s*\\#0\\s+main.*" "again back to main" +} -- 1.9.1