From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22425 invoked by alias); 29 Apr 2013 08:38:15 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 22410 invoked by uid 89); 29 Apr 2013 08:38:14 -0000 X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,MSGID_MULTIPLE_AT autolearn=no version=3.3.1 Received: from mailhost.u-strasbg.fr (HELO mailhost.u-strasbg.fr) (130.79.201.41) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Mon, 29 Apr 2013 08:38:13 +0000 Received: from md14.u-strasbg.fr (md14.u-strasbg.fr [130.79.200.249]) by mailhost.u-strasbg.fr (8.14.3/jtpda-5.5pre1) with ESMTP id r3T8c6ej008931 ; Mon, 29 Apr 2013 10:38:06 +0200 (CEST) (envelope-from pierre.muller@ics-cnrs.unistra.fr) Received: from mailserver.u-strasbg.fr (ms15.u-strasbg.fr [130.79.204.115]) by md14.u-strasbg.fr (8.14.3/jtpda-5.5pre1) with ESMTP id r3T8c5Jp026128 ; Mon, 29 Apr 2013 10:38:05 +0200 Received: from E6510Muller (gw-ics.u-strasbg.fr [130.79.210.225]) (user=mullerp mech=LOGIN) by mailserver.u-strasbg.fr (8.14.3/jtpda-5.5pre1) with ESMTP id r3T8c5ZN003887 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO) ; Mon, 29 Apr 2013 10:38:05 +0200 (envelope-from pierre.muller@ics-cnrs.unistra.fr) From: "Pierre Muller" To: "'Joel Brobecker'" , References: <20130418151631.GI3525@adacore.com> <1367212141-16536-1-git-send-email-brobecker@adacore.com> In-Reply-To: <1367212141-16536-1-git-send-email-brobecker@adacore.com> Subject: RE: [commit] Add ARI marker to get_DW_AT_signature_type. Date: Mon, 29 Apr 2013 15:30:00 -0000 Message-ID: <006f01ce44b4$dc904ed0$95b0ec70$@muller@ics-cnrs.unistra.fr> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable X-SW-Source: 2013-04/txt/msg00862.txt.bz2 Hi all, Joel, I commpletely agree with you on the principle of this patch, but I must confess that it somehow also bypasses the current implementation of the editCase rule: The problem is the following: if you lok at the patterns for "UCASE function" or "editCase function" rules, you will see that possible_editCase is set to one if and only if there is a function name starting at start of line followed by an opening brace on the same line... Which is not the case anymore after your patch... The rule is built to be able to find /* ARI: editCase function */ in any of the lines listing all the parameters.... At least, it is now that I just fixed a spelling error inside gdb_ari.sh that prevented it from working correctly before... So I would like to propose this patch: Pierre Muller as ARI maintainer. 2013-04-29 Pierre Muller * dwarf2read.c (handle_DW_AT_stmt_list): Avoid ARI warning for editCase function rule. (get_DW_AT_signature_type): Likewise. Index: dwarf2read.c =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D RCS file: /cvs/src/src/gdb/dwarf2read.c,v retrieving revision 1.790 diff -u -p -r1.790 dwarf2read.c --- dwarf2read.c 29 Apr 2013 05:05:44 -0000 1.790 +++ dwarf2read.c 29 Apr 2013 08:34:26 -0000 @@ -8053,7 +8053,7 @@ find_file_and_directory (struct die_info static void handle_DW_AT_stmt_list (struct die_info *die, struct dwarf2_cu *cu, - const char *comp_dir) + const char *comp_dir) /ARI: editCase function */ { struct attribute *attr; @@ -17924,8 +17924,8 @@ get_signatured_type (struct die_info *di reading in and processing the type unit if necessary. */ static struct type * -get_DW_AT_signature_type /* ARI: editCase */ - (struct die_info *die, struct attribute *attr, struct dwarf2_cu *cu) +get_DW_AT_signature_type (struct die_info *die, struct attribute *attr, + struct dwarf2_cu *cu) /* ARI: editCase function */ { /* Yes, DW_AT_signature can use a non-ref_sig8 reference. */ if (is_ref_attr (attr)) > -----Message d'origine----- > De=A0: gdb-patches-owner@sourceware.org [mailto:gdb-patches- > owner@sourceware.org] De la part de Joel Brobecker > Envoy=E9=A0: lundi 29 avril 2013 07:09 > =C0=A0: gdb-patches@sourceware.org > Cc=A0: Joel Brobecker > Objet=A0: [commit] Add ARI marker to get_DW_AT_signature_type. >=20 > Hello, >=20 > As discussed at: > http://www.sourceware.org/ml/gdb-patches/2013-04/msg00564.html >=20 > gdb/ChangeLog: >=20 > * dwarf2read.c (get_DW_AT_signature_type): Add ARI marker. >=20 > Tested by rebuilding GDB. Checked in. >=20 > --- > gdb/ChangeLog | 4 ++++ > gdb/dwarf2read.c | 4 ++-- > 2 files changed, 6 insertions(+), 2 deletions(-) >=20 > diff --git a/gdb/ChangeLog b/gdb/ChangeLog > index 630fd1d..429445f 100644 > --- a/gdb/ChangeLog > +++ b/gdb/ChangeLog > @@ -1,3 +1,7 @@ > +2013-04-29 Joel Brobecker > + > + * dwarf2read.c (get_DW_AT_signature_type): Add ARI marker. > + > 2013-04-28 Yao Qi >=20 > * solib-dsbt.c (fetch_loadmap): Re-indent. > diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c > index 37b7dc4..bd162e6 100644 > --- a/gdb/dwarf2read.c > +++ b/gdb/dwarf2read.c > @@ -17924,8 +17924,8 @@ get_signatured_type (struct die_info *die, ULONGEST > signature, > reading in and processing the type unit if necessary. */ >=20 > static struct type * > -get_DW_AT_signature_type (struct die_info *die, struct attribute *attr, > - struct dwarf2_cu *cu) > +get_DW_AT_signature_type /* ARI: editCase */ > + (struct die_info *die, struct attribute *attr, struct dwarf2_cu *cu) > { > /* Yes, DW_AT_signature can use a non-ref_sig8 reference. */ > if (is_ref_attr (attr)) > -- > 1.7.10.4