From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9884 invoked by alias); 24 May 2011 10:42:32 -0000 Received: (qmail 9876 invoked by uid 22791); 24 May 2011 10:42:32 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE X-Spam-Check-By: sourceware.org Received: from smtp207.alice.it (HELO smtp207.alice.it) (82.57.200.103) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 24 May 2011 10:41:59 +0000 Received: from [192.168.1.4] (79.45.212.220) by smtp207.alice.it (8.5.124.08) id 4DB138870312FB89; Tue, 24 May 2011 12:41:54 +0200 Message-ID: <4DDB8BA5.60507@oracle.com> Date: Tue, 24 May 2011 11:50:00 -0000 From: Paolo Carlini User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110414 SUSE/3.1.10 Thunderbird/3.1.10 MIME-Version: 1.0 To: Jason Merrill CC: gcc-patches List Subject: Re: RFA (libstdc++): C++/v3 PATCH for c++/24163 (lookup in dependent bases) and c++/29131 References: <4DD69790.8070101@redhat.com> <4DDB2D1F.1010305@redhat.com> In-Reply-To: <4DDB2D1F.1010305@redhat.com> Content-Type: multipart/mixed; boundary="------------070106070201060501050204" X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-05/txt/msg01711.txt.bz2 This is a multi-part message in MIME format. --------------070106070201060501050204 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-length: 286 On 05/24/2011 05:59 AM, Jason Merrill wrote: > It occurred to me today that I could use current_class_name rather > than TYPE_IDENTIFIER (current_class_type). Thus I suppose the below is also ok and obvious? To be safe, I'm testing it on x86_64-linux. Paolo. ////////////////////// --------------070106070201060501050204 Content-Type: text/plain; name="CL" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="CL" Content-length: 107 2011-05-24 Paolo Carlini * decl.c (grokdeclarator): Use current_class_name. --------------070106070201060501050204 Content-Type: text/plain; name="patchlet" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="patchlet" Content-length: 556 Index: decl.c =================================================================== --- decl.c (revision 174105) +++ decl.c (working copy) @@ -9910,7 +9910,7 @@ grokdeclarator (const cp_declarator *declarator, instantiation made the field's type be incomplete. */ if (current_class_type && TYPE_NAME (current_class_type) - && IDENTIFIER_TEMPLATE (TYPE_IDENTIFIER (current_class_type)) + && IDENTIFIER_TEMPLATE (current_class_name) && declspecs->type && declspecs->type == type) error (" in instantiation of template %qT", --------------070106070201060501050204--