From patchwork Fri Nov 29 10:25:21 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Simon Marchi (Code Review)" X-Patchwork-Id: 36372 Received: (qmail 11157 invoked by alias); 29 Nov 2019 10:25: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 11098 invoked by uid 89); 29 Nov 2019 10:25:28 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3 autolearn=ham version=3.3.1 spammy=baris, tankutbarisaktemurintelcom, tankut, sk:tankut X-HELO: mx1.osci.io Received: from polly.osci.io (HELO mx1.osci.io) (8.43.85.229) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 29 Nov 2019 10:25:26 +0000 Received: by mx1.osci.io (Postfix, from userid 994) id 9389A20456; Fri, 29 Nov 2019 05:25:22 -0500 (EST) Received: from gnutoolchain-gerrit.osci.io (gnutoolchain-gerrit.osci.io [IPv6:2620:52:3:1:5054:ff:fe06:16ca]) by mx1.osci.io (Postfix) with ESMTP id 7433B20387 for ; Fri, 29 Nov 2019 05:25:21 -0500 (EST) Received: from localhost (localhost [127.0.0.1]) by gnutoolchain-gerrit.osci.io (Postfix) with ESMTP id 528AA28173 for ; Fri, 29 Nov 2019 05:25:21 -0500 (EST) X-Gerrit-PatchSet: 1 Date: Fri, 29 Nov 2019 05:25:21 -0500 From: "Tankut Baris Aktemur (Code Review)" To: gdb-patches@sourceware.org Message-ID: Auto-Submitted: auto-generated X-Gerrit-MessageType: newchange Subject: [review] gdb: improve debug output of function overload resolution X-Gerrit-Change-Id: I771017e7afbbaf4809e2238a9b23274f55c61f55 X-Gerrit-Change-Number: 735 X-Gerrit-ChangeURL: X-Gerrit-Commit: 480cae6f559264978618a2ae28911f3937eb32ac References: Reply-To: tankut.baris.aktemur@intel.com, gdb-patches@sourceware.org MIME-Version: 1.0 Content-Disposition: inline User-Agent: Gerrit/3.0.3-79-g83ff7f88f1 Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/735 ...................................................................... gdb: improve debug output of function overload resolution Function overload resolution prints debug output if turned on via the 'set debug overload' command. The output includes the badness vector (BV). For each function, this vector contains a badness value of the length of parameters as its first element. So, BV[0] does not correspond to a parameter. The badness values of parameters start with BV[1]. A badness value is a pair; it contains a rank and a subrank. Printing both fields provides useful information. Improve printing the badness vector along these lines. gdb/ChangeLog: 2019-11-29 Tankut Baris Aktemur * valops.c (find_oload_champ): Improve debug output. Change-Id: I771017e7afbbaf4809e2238a9b23274f55c61f55 --- M gdb/valops.c 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/gdb/valops.c b/gdb/valops.c index 8af53de..e3fc2dc 100644 --- a/gdb/valops.c +++ b/gdb/valops.c @@ -3039,10 +3039,15 @@ "%s # of parms %d\n", functions[ix]->demangled_name (), (int) parm_types.size ()); - for (jj = 0; jj < args.size () - static_offset; jj++) + + fprintf_filtered (gdb_stderr, + "...Badness of length : {%d, %d}\n", + bv[0].rank, bv[0].subrank); + + for (jj = 1; jj < bv.size (); jj++) fprintf_filtered (gdb_stderr, - "...Badness @ %d : %d\n", - jj, bv[jj].rank); + "...Badness of arg %d : {%d, %d}\n", + jj, bv[jj].rank, bv[jj].subrank); } if (oload_champ_bv->empty ())