From patchwork Tue Nov 22 14:39:12 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom de Vries X-Patchwork-Id: 127089 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id 5838DB6EE8 for ; Wed, 23 Nov 2011 01:39:37 +1100 (EST) Received: (qmail 16738 invoked by alias); 22 Nov 2011 14:39:35 -0000 Received: (qmail 16713 invoked by uid 22791); 22 Nov 2011 14:39:34 -0000 X-SWARE-Spam-Status: No, hits=-1.8 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 22 Nov 2011 14:39:21 +0000 Received: from svr-orw-fem-01.mgc.mentorg.com ([147.34.98.93]) by relay1.mentorg.com with esmtp id 1RSrVI-0001d6-RT from Tom_deVries@mentor.com ; Tue, 22 Nov 2011 06:39:20 -0800 Received: from SVR-IES-FEM-01.mgc.mentorg.com ([137.202.0.104]) by svr-orw-fem-01.mgc.mentorg.com over TLS secured channel with Microsoft SMTPSVC(6.0.3790.4675); Tue, 22 Nov 2011 06:39:20 -0800 Received: from [127.0.0.1] (137.202.0.76) by SVR-IES-FEM-01.mgc.mentorg.com (137.202.0.104) with Microsoft SMTP Server id 14.1.289.1; Tue, 22 Nov 2011 14:39:18 +0000 Message-ID: <4ECBB410.8020906@mentor.com> Date: Tue, 22 Nov 2011 15:39:12 +0100 From: Tom de Vries User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.23) Gecko/20110922 Lightning/1.0b2 Thunderbird/3.1.15 MIME-Version: 1.0 To: Richard Guenther CC: "gcc-patches@gcc.gnu.org" Subject: [PATCH] fix DO_STMT comment in cp-tree.def. Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Richard, The comment at DO_STMT led me to believe that DO_BODY was the first operand. cp-tree.def: ... ./* Used to represent a 'do' statement. The operands are DO_BODY and DO_COND, respectively. */ DEFTREECODE (DO_STMT, "do_stmt", tcc_statement, 2) .. However, in cp-tree.h we see that DO_BODY is actually the second operand: ... /* DO_STMT accessors. These give access to the condition of the do statement and the body of the do statement, respectively. */ #define DO_COND(NODE) TREE_OPERAND (DO_STMT_CHECK (NODE), 0) #define DO_BODY(NODE) TREE_OPERAND (DO_STMT_CHECK (NODE), 1) ... The patch fixes the comment. OK for trunk? Thanks, - Tom 2011-11-22 Tom de Vries * cp-tree.def (DO_STMT): Fix comment. Index: gcc/cp/cp-tree.def =================================================================== --- gcc/cp/cp-tree.def (revision 181172) +++ gcc/cp/cp-tree.def (working copy) @@ -308,8 +308,8 @@ DEFTREECODE (RANGE_FOR_STMT, "range_for_ and WHILE_BODY, respectively. */ DEFTREECODE (WHILE_STMT, "while_stmt", tcc_statement, 2) -/* Used to represent a 'do' statement. The operands are DO_BODY and - DO_COND, respectively. */ +/* Used to represent a 'do' statement. The operands are DO_COND and + DO_BODY, respectively. */ DEFTREECODE (DO_STMT, "do_stmt", tcc_statement, 2) /* Used to represent a 'break' statement. */