From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 108571 invoked by alias); 3 Apr 2015 21:10:02 -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 108561 invoked by uid 89); 3 Apr 2015 21:10:01 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL,BAYES_00,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: cam-smtp0.cambridge.arm.com Received: from fw-tnat.cambridge.arm.com (HELO cam-smtp0.cambridge.arm.com) (217.140.96.140) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Fri, 03 Apr 2015 21:10:00 +0000 Received: from arm.com (e106375-lin.cambridge.arm.com [10.2.206.37]) by cam-smtp0.cambridge.arm.com (8.13.8/8.13.8) with ESMTP id t33L9uNK003757; Fri, 3 Apr 2015 22:09:56 +0100 Date: Fri, 03 Apr 2015 21:10:00 -0000 From: James Greenhalgh To: Ramana Radhakrishnan Cc: Sebastian Pop , Evandro Menezes , Kyrylo Tkachov , GCC Patches Subject: Re: [PATCH] [ARM] Add support for the Samsung Exynos M1 processor Message-ID: <20150403210956.GA5607@arm.com> References: <055b01d06b33$bdcce3d0$3966ab70$@samsung.com> <551A5BCA.1030203@arm.com> <003601d06c13$203447e0$609cd7a0$@samsung.com> <20150402225106.GA18324@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-IsSubscribed: yes X-SW-Source: 2015-04/txt/msg00149.txt.bz2 On Fri, Apr 03, 2015 at 07:53:12PM +0100, Ramana Radhakrishnan wrote: > On Fri, Apr 3, 2015 at 5:17 PM, Sebastian Pop wrote: > > Hi, > > > > On Thu, Apr 2, 2015 at 5:51 PM, James Greenhalgh > > wrote: > >> Trunk is currently in Stage 4 development, these patches are fairly > >> low-risk, but they are certainly not regression fixes. I'll defer > >> to port maintainers and release managers for the final say, but in my > >> opinion it would not be appropriate to commit them until Stage 1 > >> development for GCC 6.0 opens (hopefully in a few weeks). > > > > I thought that adding flags for new processors was ok at any time, > > even to backport. > > It's usually risk vs reward on a per patch basis and I don't think of > it as a general rule. We've always avoided the CPU tuning backport > rule to the FSF branches. The smaller the CPU tuning patch - the > better it is and in this case I'm comfortable with the patch going in > as it is adding another tuning option, using existing constructs and > is not invasive in the backend. Thanks for the clarification Ramana. In which case, and now that I've seen that binutils support has also been accepted, the AArch64 part is OK to commit (assuming no regressions and no objections from Richard or Jakub). It would be great if you could follow these up with a patch for changes.html for GCC 5 for both ARM and AArch64. Cheers, James