From patchwork Tue Nov 14 17:08:13 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Marchi X-Patchwork-Id: 24239 Received: (qmail 118795 invoked by alias); 14 Nov 2017 17:08:31 -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 118786 invoked by uid 89); 14 Nov 2017 17:08:30 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-23.9 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, KAM_STOCKGEN, KB_WAM_FROM_NAME_SINGLEWORD autolearn=ham version=3.3.2 spammy= X-HELO: sessmg23.ericsson.net Received: from sessmg23.ericsson.net (HELO sessmg23.ericsson.net) (193.180.251.45) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 14 Nov 2017 17:08:28 +0000 Received: from ESESSHC010.ericsson.se (Unknown_Domain [153.88.183.48]) by sessmg23.ericsson.net (Symantec Mail Security) with SMTP id 9B.4B.07741.9032B0A5; Tue, 14 Nov 2017 18:08:25 +0100 (CET) Received: from EUR02-VE1-obe.outbound.protection.outlook.com (153.88.183.145) by oa.msg.ericsson.com (153.88.183.48) with Microsoft SMTP Server (TLS) id 14.3.352.0; Tue, 14 Nov 2017 18:08:24 +0100 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=simon.marchi@ericsson.com; Received: from elxacz23q12.ca.am.ericsson.se (192.75.88.130) by AMSPR07MB312.eurprd07.prod.outlook.com (2a01:111:e400:802f::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.20.239.4; Tue, 14 Nov 2017 17:08:22 +0000 From: Simon Marchi To: CC: Simon Marchi Subject: [PATCH] Show optimized out local variables in "info locals" Date: Tue, 14 Nov 2017 12:08:13 -0500 Message-ID: <1510679293-8244-1-git-send-email-simon.marchi@ericsson.com> MIME-Version: 1.0 X-ClientProxiedBy: BN6PR16CA0010.namprd16.prod.outlook.com (2603:10b6:404:f5::20) To AMSPR07MB312.eurprd07.prod.outlook.com (2a01:111:e400:802f::24) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: fbede8f9-db1d-4389-06b3-08d52b825027 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(4534020)(4602075)(4627115)(201703031133081)(201702281549075)(2017052603258); SRVR:AMSPR07MB312; X-Microsoft-Exchange-Diagnostics: 1; AMSPR07MB312; 3:g/2UMHdMRxCISzoNSSPchPBnOLW4nas7/Wj55RiFMSLXZ4Y6p3PI0DXpsrhIRhwzWSSOCg+IzQ/klz2kYH3Bt2IEkbIfD3QBayVQbDVuXWwbre9UzNgIKdTAoLdYArwA/1J5lwbMhjZTvpAqse2AUhIIshtFNMeMXEWuxXgHt/cC5vN2r7Pl+QSNfmVyn7Nz5/VktyAH6dHKTgjAuUSlqK4GpZG00yvnIr7Gr5h8PZKDvoEuWl3HFmwciuBbnohX; 25:X76Jx8sYXrhZA5A7iAotUoPIY/JFc9pfKifXGiA7Uw15wmLldimyaeP9iDyyZeE3uFWBvSG9YTb9HiDDLxzOnLbZK8eetJtTQ/wHpgiQqMrNtLgpfVrvXhmr7+KQ+O17Kw6FQatROR9JsmPpg5C771b0DU1C9OFg741Qa/os5lPxravkLdIYYkR5YVTwhGN/mu42zFJJ1m1M7RNcFSNZNlt6LbyTkobUTHP6dvBCfuuXzIiTI+ydZHZvOBp95KHq8QQBSzHkHrjh5X9pgnkb9j1+5BU9ZFKOeJbf64DIjRU4gJ60EYg0DmJtfQGGLMbWWrQH9u/jF1ioNaB6DhroJdJRw/sfPd2Eeyefqtd21qs=; 31:BtX+92bdQ0icr4A/1Ky17F1ya1lxqt31G/8kAyhskOMMqidxp19Z/5kjkCOXvSXtzvLskQvegkWjI3Ph8UnEqpXAIXzTlqORYcEE4WPdZ9sFVkhitjVGw393w9wI+4PTK0pbZUGS6T3RRk3j9Gq3nVsZ1xT6hxdjO7wsOuHRwgmEi22u+Z80EGRA41XEww6+GktxwaOGsLW3TI1PDKXek11D5aqnb1S7mWQ7mXT6aZ4= X-MS-TrafficTypeDiagnostic: AMSPR07MB312: X-Microsoft-Exchange-Diagnostics: 1; AMSPR07MB312; 20:0NEp9DjBeL44julQu2e+2jBMJQjZGfRH3grvPpysNEVc/rsIL8CMqYhzGdmikZd686u9Kmt/m3vTQRH/YnOrlFyyO5s9/QsDfA4L5O8bkz7zRaPkmO5kUPJbxTLJro9yIchyqMO9MMiF7PXwEosHyyu+0hEaAjyy4OK0DGlxuRdOXQwlR66oU5TUG/GesrxJxtwGr3ifUxj6JP5AoZN/mYi7eD95hRwCwWNK7hzhH8AH3lofeS0Of/LXc3vHSdP+vxJji27HdzBpJ34a8s3pPZKY3QDzyneP2mxk5MBmqVIGLu49rS+O/YDYcsHhav0xf99uecLFpXf15x8XHIAKPFQ34N/vBvYaEIOTToeJ8zr20nuLOQ3HVRm0o+C/2RKd2SkTk7C3P7X3VSUM91JZd89YcE0d3vZZDXFR4HpseZK07eDqsrw4ZC3t+4v89yhVY17gHKBBHuhWorKl2+X2Jw0l4ypn0Thi6PqV6cXAHrJUhYUW+0UJ8BdAua+XQ2r3; 4:CW9wj8MV1k18K0+PoYE0ye9SngAcTp/915vnoCw4bsJFHr3H0tn3u4p3sJVkaU4ypcL9+G0x7M8yjXwsRcQdtH8oX+1lOPG1GgWiUYnLY1KuIZxqiTbaGgmuD/6LXk1LZX2RLqC2gqzAjBHEFHOx+jYvoIXl/JqavruQS6TRMgHgHAcLdOEs+jqoHtELo5AgkU5Djvd+CZpaiSqx/KdgqtCZoVwJhst1if0OyGdYzjN8CEZ9+vYkMYadBAOdHVs5mV/X02D/Ts0Fz4avOQF+/tyXt//ir/T6PXM2LOWC7daXGUhCz/X+Q1AMLP22RbjiAMLdLhc9pUX+keKBKyEPJd95QfHQSLpCp8pkIttbuBY= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(250305191791016)(22074186197030); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(3231022)(100000703101)(100105400095)(93006095)(93001095)(3002001)(10201501046)(6041248)(20161123562025)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123560025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:AMSPR07MB312; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:AMSPR07MB312; X-Forefront-PRVS: 04916EA04C X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(39860400002)(346002)(376002)(54534003)(199003)(189002)(97736004)(189998001)(6666003)(7736002)(2351001)(6306002)(6512007)(6916009)(305945005)(86362001)(2361001)(53936002)(3846002)(6116002)(5660300001)(105586002)(106356001)(81156014)(81166006)(6506006)(16526018)(8676002)(8936002)(68736007)(50466002)(50226002)(33646002)(50986999)(6486002)(5003940100001)(575784001)(107886003)(316002)(966005)(16586007)(48376002)(25786009)(47776003)(478600001)(66066001)(2906002)(101416001)(36756003)(4326008)(2004002); DIR:OUT; SFP:1101; SCL:1; SRVR:AMSPR07MB312; H:elxacz23q12.ca.am.ericsson.se; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: ericsson.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; AMSPR07MB312; 23:t+7bE7v9Lu6nlwY58zgyKhGIfme0M/bvOClsEJ2gNd?= =?us-ascii?Q?C7xPIF0p06ZJeIJRjzI7jPhXtYXXkKCM/pntY/R/1B5xtZSPrkXd5+Cqg8Gj?= =?us-ascii?Q?sV5pZZjs5D0C7/E2+LOEny9sKPs/a4UE2Xogm/slO3ALn+c90GyFeQCFAKa/?= =?us-ascii?Q?coYJoD7oTQUpSSL2HUQ7D0jURxrHAImDfR+z0TxE8XaA3CZCUVOvu8/p5otG?= =?us-ascii?Q?ytOfRTA5tdUd6bCKW1eLBLnicwrBLEmBYeJ45991Gp1aRhgiWpe3elLqQXni?= =?us-ascii?Q?K/jqCn9mYpJYJCDSr+kulBndbj3dfW+qQvz1yhb3uS+ssTYZFlKMgEr6qbFg?= =?us-ascii?Q?9hELKvaXUg1aAgBKwocXSCUiXV+V4khRsd9OBojvPINUfONKfYFAHjmFt1v4?= =?us-ascii?Q?bkINgh/QOno3hAzDG/WdM4W92szpG1fVX4EzGP/utAT+g7v6uoekdLOu0PU+?= =?us-ascii?Q?+nBBzP/yamPYQSj4QNRyeae/FynczofmHpLo4c3/rU6OoFgjGWeAVsKFLDyh?= =?us-ascii?Q?gn9iMnDsS/5hqMTeJpIbY40GAVdtbLGCWKh5IbUNWo0b4mrYouIPNbnPwBWP?= =?us-ascii?Q?OKMW86XJsofaXt3tQmRahXiDdo2DfhC0juU+QJqCb4hSgvuvaEfo1RDn5BEd?= =?us-ascii?Q?6jFWMh0WFpiLGRWk7afBGtcUKSvqrURvjTniGuqminwwSFy64cxfCc+YVfo2?= =?us-ascii?Q?cAbB7ZYE3qo1zZMndHr2LPneymUSV5R/6LZj7PDwE1K0YPDyypDzvAUTpGYt?= =?us-ascii?Q?dO6F4qCoUoXIzGekRzrDKma+Q/N0l+qhANouip2VPG/d4nQEzoAUOiSlxbRu?= =?us-ascii?Q?swye2dEmZwKhbEHNP0PgcNdYIRNimegV8UAz9AWp7ACUKBuWylmn/7SKM6Sw?= =?us-ascii?Q?3wLD93L4ZR9yCkC741ficGY8ajFZ0+THk3uRNcKdD7J4zKt3rYkR3g5/pI8Y?= =?us-ascii?Q?Or+DdFNCiK9t1MClSovmFFSiW3Re1chkH/aKVV0T4EBkLzh/b0hMi/5K+Byk?= =?us-ascii?Q?IGlljX/Jx82NAc3KBejVoOUKBuQz42J60A5RDZKG5PPKsYGTCRu7sag5Yav7?= =?us-ascii?Q?yaz/Ei+xktM+RR6BcH2N8/BeZlXEl7lUKW5YEhqFMxp+0Ap4kkQVMEuhQO0O?= =?us-ascii?Q?gYo8xhve0GVAS7WUf//UYP5b7v+KS3ggeVJuArChQs79h8kydngzIrumEH6W?= =?us-ascii?Q?xzxMZX5TrThqtvNBt5+aDv8/2Zd6A+2LdH?= X-Microsoft-Exchange-Diagnostics: 1; AMSPR07MB312; 6:KysFzkydMWkvXGvTO8/75hGD8NRuXiMqNFcX3sYxd1R6wx5gwmn2gA4bjdk7vtdzq2UtHBeFTEwn1g0w1GGPazwYmhLpB1PF/DBrbM8aqdtc/bTCAJ+QyU+7YrkBRcQRZJTvU4o14yk2Z/Uq/6mGVNMjoMxQyG9nBdnAUWpjoaaI7DdG0XWwK6acn214vnAk5isSC1a0s5mVuUruuYokOhCcZl/03xGE5TPCzAj73KOtsF0Z7IDkDyX+1gwVqfu2Ci1fM1BVgWMWXhBGlvCxXtfclyS/L/6hJAseXSAfBvEp5bObLMU/5/XdGiKDK2hE5x4W9jLixtxRH1Agdnaf4pnT1p25ThOAtRYt7rcLxAM=; 5:/HesUBfeOL3TZH+MVpvygy+YrEXFoDBFD/36yua0FavM3VIt4Ofqb/lz95iIPi+NRc4Sgk7ceiqLSTzGZNtWt4a0UFTYznT4oiOvxZ9+DnYSXCvpP2zhKuwF+BhH7PA48p1P7zb5zNfhSzKKe7lDeuNPLzKgJbtTr7RA7Qcr+RE=; 24:I/CtFy1q6stLVLEQZEW9ZrYvdSCbnhKFnGiLmZWgbbuUggsdAgkVEFwAjLxiRA4fBylfBQ+Ej1RtMht6C1Rw6nPnU5sKPAX1lB4obAJ1bKU=; 7:TvQmw4X7z3OUjE2IntMohmGwghINtc8xXXLRoxgD08RCOguattJWeH1FwFsT/7fOrmo6N7qHolf6ASeimgTib/ZadvqObdhJBbeQiE8edbUZnmUfLS7lBecsoNDF95GUDV9CkohI3zuvEJGXOt7cGo0SrcS1syNjy70SAoht9lWqTO1tipzCpmIiB0uB/rFOpYAqLqfO/dj+OIkiIM9rEmXKZx44Nlko1/yca4lbbJWqcxsV/G/bpzuP0gmy04Ph SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Nov 2017 17:08:22.8895 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: fbede8f9-db1d-4389-06b3-08d52b825027 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 92e84ceb-fbfd-47ab-be52-080c6b87953f X-MS-Exchange-Transport-CrossTenantHeadersStamped: AMSPR07MB312 X-OriginatorOrg: ericsson.com X-IsSubscribed: yes Currently, optimized out variables are not shown when doing "info locals". Some users found that confusing, thinking GDB forgot to print their variable. This patch adds them to the "info locals" output. I added a test in gdb.dwarf2 to test for that behavior. I think doing a synthetic DWARF test is the easiest way to have an optimized out local variable for sure. However, this change reveals what I think is a bug in GDB, see: http://lists.dwarfstd.org/pipermail/dwarf-discuss-dwarfstd.org/2017-September/004394.html This patch marks the tests in inline-locals.exp that start failing as KFAIL. I'd like to tackle this bug eventually, but I don't have the time right now. I think it's still better to show an extra erroneous entry than to not show the optimized out variables at all. I haven't created a bug in bugzilla yet, but if we agree it's indeed a bug, I'll create one and update the setup_kfail lines with the actual bug number before pushing. gdb/ChangeLog: * stack.c (iterate_over_block_locals): Add LOC_OPTIMIZED_OUT case in switch. gdb/testsuite/ChangeLog: * gdb.opt/inline-locals.exp: Mark tests as KFAIL. * gdb.dwarf2/info-locals-optimized-out.exp: New file. * gdb.dwarf2/info-locals-optimized-out.c: New file. --- gdb/stack.c | 1 + .../gdb.dwarf2/info-locals-optimized-out.c | 23 +++++++ .../gdb.dwarf2/info-locals-optimized-out.exp | 74 ++++++++++++++++++++++ gdb/testsuite/gdb.opt/inline-locals.exp | 2 + 4 files changed, 100 insertions(+) create mode 100644 gdb/testsuite/gdb.dwarf2/info-locals-optimized-out.c create mode 100644 gdb/testsuite/gdb.dwarf2/info-locals-optimized-out.exp diff --git a/gdb/stack.c b/gdb/stack.c index 81032fc..6bd0d45 100644 --- a/gdb/stack.c +++ b/gdb/stack.c @@ -1922,6 +1922,7 @@ iterate_over_block_locals (const struct block *b, case LOC_REGISTER: case LOC_STATIC: case LOC_COMPUTED: + case LOC_OPTIMIZED_OUT: if (SYMBOL_IS_ARGUMENT (sym)) break; if (SYMBOL_DOMAIN (sym) == COMMON_BLOCK_DOMAIN) diff --git a/gdb/testsuite/gdb.dwarf2/info-locals-optimized-out.c b/gdb/testsuite/gdb.dwarf2/info-locals-optimized-out.c new file mode 100644 index 0000000..e3ae4cd --- /dev/null +++ b/gdb/testsuite/gdb.dwarf2/info-locals-optimized-out.c @@ -0,0 +1,23 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2017 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 . */ + +int +main (void) +{ + asm ("main_label: .globl main_label"); + return 0; +} diff --git a/gdb/testsuite/gdb.dwarf2/info-locals-optimized-out.exp b/gdb/testsuite/gdb.dwarf2/info-locals-optimized-out.exp new file mode 100644 index 0000000..dd0a99b --- /dev/null +++ b/gdb/testsuite/gdb.dwarf2/info-locals-optimized-out.exp @@ -0,0 +1,74 @@ +# Copyright 2017 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 . + +# Test that "info locals" shows optimized out variables. + +load_lib dwarf.exp + +# This test can only be run on targets which support DWARF-2 and use gas. +if {![dwarf2_support]} { + return 0 +} + +standard_testfile .c ${gdb_test_file_name}-dw.S + +# Make some DWARF for the test. + +set asm_file [standard_output_file $srcfile2] +Dwarf::assemble $asm_file { + global srcdir subdir srcfile + global dwarf_regnum regname + + set buf_var [gdb_target_symbol buf] + + cu {} { + DW_TAG_compile_unit { + {DW_AT_name info-locals-optimized-out.c} + {DW_AT_comp_dir /tmp} + } { + declare_labels int_type_label + + # int + int_type_label: base_type { + {name "int"} + {encoding @DW_ATE_signed} + {byte_size 4 DW_FORM_sdata} + } + + DW_TAG_subprogram { + {MACRO_AT_func { main ${srcdir}/${subdir}/${srcfile} }} + {DW_AT_external 1 flag} + } { + # A variable completely optimized out. + DW_TAG_variable { + {name "opt_out"} + {type :$int_type_label} + } + } + } + } +} + +if { [prepare_for_testing ${testfile}.exp ${testfile} \ + [list $srcfile $asm_file] {nodebug}] } { + return -1 +} + +if ![runto_main] { + return -1 +} + +# Make sure "info locals" shows optimized out variables. +gdb_test "info locals" ".*opt_out = .*" "info local shows optimized out variable" diff --git a/gdb/testsuite/gdb.opt/inline-locals.exp b/gdb/testsuite/gdb.opt/inline-locals.exp index 76463a6..7245df7 100644 --- a/gdb/testsuite/gdb.opt/inline-locals.exp +++ b/gdb/testsuite/gdb.opt/inline-locals.exp @@ -43,6 +43,7 @@ if { ! $no_frames } { "backtrace from bar 2" gdb_test "up" "#1 .*func1 .* at .*" "up from bar 2" gdb_test "info frame" ".*inlined into frame.*" "func1 inlined 2" + setup_kfail "gdb/xyz" *-*-* gdb_test "info locals" "array = {.*}" "info locals above bar 2" set msg "info args above bar 2" @@ -82,6 +83,7 @@ if { ! $no_frames } { "backtrace from bar 3" gdb_test "up" "#1 .*func1 .* at .*" "up from bar 3" gdb_test "info frame" ".*inlined into frame.*" "func1 inlined 3" + setup_kfail "gdb/xyz" *-*-* gdb_test "info locals" "array = {.*}" "info locals above bar 3" set msg "info args above bar 3"