From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32111 invoked by alias); 11 Aug 2015 11:22:36 -0000 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 Received: (qmail 32097 invoked by uid 89); 11 Aug 2015 11:22:36 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.7 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.2 X-HELO: eu-smtp-delivery-143.mimecast.com Received: from eu-smtp-delivery-143.mimecast.com (HELO eu-smtp-delivery-143.mimecast.com) (146.101.78.143) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 11 Aug 2015 11:22:35 +0000 Received: from cam-owa2.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.140]) by eu-smtp-1.mimecast.com with ESMTP id uk-mta-33-ooNjWspcRkuQWE19UZStBQ-1; Tue, 11 Aug 2015 12:22:29 +0100 Received: from e104437-lin ([10.1.2.79]) by cam-owa2.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Tue, 11 Aug 2015 12:22:29 +0100 References: <20150804094851.GA12175@arm.com> <87io8q6le7.fsf@igel.home> From: Jiong Wang To: Andreas Schwab Cc: James Greenhalgh , gcc-patches Subject: [COMMITTED][AArch64] Add the missing "," for enumeration element Date: Tue, 11 Aug 2015 11:22:00 -0000 In-reply-to: Message-ID: MIME-Version: 1.0 X-MC-Unique: ooNjWspcRkuQWE19UZStBQ-1 Content-Type: multipart/mixed; boundary="=-=-=" X-SW-Source: 2015-08/txt/msg00507.txt.bz2 --=-=-= Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable Content-length: 969 Jiong Wang writes: > Andreas Schwab writes: > >> Jiong Wang writes: >> >>> I Just finished several round of rebuild & testing on clean >>> environment. >> >> How did you even manage to compile it? >> >> ../../gcc/ira.c: In function 'void print_translated_classes(FILE*, bool)= ': >> ../../gcc/ira.c:1415:49: error: iteration 8u invokes undefined behavior = [-Werror=3Daggressive-loop-optimizations] >> fprintf (f, " %s -> %s\n", reg_class_names[i], >> ^ >> ../../gcc/ira.c:1414:17: note: containing loop >> for (i =3D 0; i < N_REG_CLASSES; i++) > > And, my new patch missed a ',' after "FIXED_REG0", sorry about > that.. that should be the problem for the bootstrap issue. committed patch below as obivious after bootstrap OK and check-go OK. 2015-08-11 Jiong Wang gcc/ * config/aarch64/aarch64.h (REG_CLASS_NAMES): Add the missing ',' after FIXED_REG0. --=-=-= Content-Type: text/x-diff; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline; filename=k.patch Content-length: 387 diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h index 43ff895..d3ae393 100644 --- a/gcc/config/aarch64/aarch64.h +++ b/gcc/config/aarch64/aarch64.h @@ -417,7 +417,7 @@ enum reg_class #define REG_CLASS_NAMES \ { \ "NO_REGS", \ - "FIXED_REG0" \ + "FIXED_REG0", \ "CALLER_SAVE_REGS", \ "GENERAL_REGS", \ "STACK_REG", \ --=-=-=--