From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 39189 invoked by alias); 21 Jun 2016 14:16:58 -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 39106 invoked by uid 89); 21 Jun 2016 14:16:57 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy=kindly X-HELO: mail-it0-f46.google.com Received: from mail-it0-f46.google.com (HELO mail-it0-f46.google.com) (209.85.214.46) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Tue, 21 Jun 2016 14:16:42 +0000 Received: by mail-it0-f46.google.com with SMTP id g127so19152255ith.0 for ; Tue, 21 Jun 2016 07:16:41 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=OdjSQ2wvYl0+t6LCKZkD6ww4ZlwVFhPM+yJkvMNvdZU=; b=HR/dm3eKiYCjMuIeXtZNkPA/usWqNV5qGx5IX7ZEL9pFEjnVMTAEz3G6nRvlMHXHX5 v0YbAIUgL1AxEjfZgyOzPFLdEsk1KCNygcXjQjl+zSFbw8kutF8/plXWcBz5Tw7bb5ts ctz2WBFRP+FsPSshFQpD/oeAmHAk7Nyskg+ANjImpl4w0kUl9rhfn8FNPn4McOBM872i TzK2whgMOKApFvrJ9ZoTfAE+Rmnb9dMXZWZaYo30sPnf9z7+U5B82e7oTI3GAX3W3VVV s2P3y7U/eoRSJ59ACdfRFrORaSZUYXfZehiHi6rCfSU46e7Ot+mILbegrNTtDaKlzkxg Glgg== X-Gm-Message-State: ALyK8tKQyOBrvnbRbKr8arKgIy3iX2XdeinO+FqcN1u4vX39TCjd6tLUpOgHexwUkuVXRi/GObBINuN0Qt1UO5a8 X-Received: by 10.36.131.71 with SMTP id d68mr5714670ite.35.1466518599762; Tue, 21 Jun 2016 07:16:39 -0700 (PDT) MIME-Version: 1.0 Received: by 10.50.202.106 with HTTP; Tue, 21 Jun 2016 07:16:20 -0700 (PDT) In-Reply-To: <20160621134640.GA31439@arm.com> References: <20160621134640.GA31439@arm.com> From: Virendra Pathak Date: Tue, 21 Jun 2016 14:16:00 -0000 Message-ID: Subject: Re: [PATCH/AARCH64] Accept vulcan as a cpu name for the AArch64 port of GCC To: James Greenhalgh Cc: gcc-patches@gcc.gnu.org, Marcus Shawcroft , Richard Earnshaw , nd@arm.com Content-Type: text/plain; charset=UTF-8 X-SW-Source: 2016-06/txt/msg01495.txt.bz2 Hi James, > This patch is OK for trunk. Thank you for the review and merging the patch to trunk. > I couldn't spot your name in the MAINTAINERS file, so I've applied this > on your behalf as revision 237645. My name is not present in the MAINTAINERS file. This was my first patch in the GCC :-) In future I will request aarch64's maintainers to apply the patches on my behalf after review (to avoid any confusion). Thanks. with regards, Virendra Pathak On Tue, Jun 21, 2016 at 7:16 PM, James Greenhalgh wrote: > > On Sat, Jun 18, 2016 at 01:57:43AM +0530, Virendra Pathak wrote: > > Hi, > > > > Please find the patch for introducing vulcan as a cpu name for the > > AArch64 port of GCC. > > Broadcom's vulcan is an armv8.1-a aarch64 server processor. > > > > Since vulcan is the first armv8.1-a processor to be introduced in > > aarch64-cores.def, > > I have created a new section in the file for the armv8.1 based processors. > > Kindly let me know if that is okay. > > > > Tested the patch with cross aarch64-linux-gnu, bootstrapped native > > aarch64-unknown-linux-gnu > > and make check (gcc, ld, gas, binutils, gdb). > > No new regression failure is added by this patch. > > > > In addition, tested -mcpu=vulcan -mtune=vulcan flags by passing them > > via command line. > > Also verified that above flags passes armv8.1-a option to assembler(as). > > > > At present we are using schedule & cost model of cortex-a57 but > > soon we will be submitting one for vulcan. > > > > Please review the patch. > > Ok for trunk? > > > > This patch is OK for trunk. > > I couldn't spot your name in the MAINTAINERS file, so I've applied this > on your behalf as revision 237645. > > Thank you for the patch. > > Thanks, > James > > > gcc/ChangeLog: > > > > Virendra Pathak > > > > * config/aarch64/aarch64-cores.def (vulcan): New core. > > * config/aarch64/aarch64-tune.md: Regenerate. > > * doc/invoke.texi: Document vulcan as an available option. > > >