From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by sourceware.org (Postfix) with ESMTP id 47C7C3858C74 for ; Tue, 17 May 2022 09:56:22 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 47C7C3858C74 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 173DD1042; Tue, 17 May 2022 02:56:22 -0700 (PDT) Received: from [10.57.5.137] (unknown [10.57.5.137]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 305A83F66F; Tue, 17 May 2022 02:56:21 -0700 (PDT) Message-ID: Date: Tue, 17 May 2022 10:56:19 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 Subject: Re: [PATCH][Binutils]AArch64 Enable FP16 by default for Armv9-A. Content-Language: en-GB To: Tamar Christina , binutils@sourceware.org Cc: Richard.Earnshaw@arm.com, nd@arm.com, marcus.shawcroft@arm.com References: From: Richard Earnshaw In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-3498.1 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, NICE_REPLY_A, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: binutils@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Binutils mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 May 2022 09:56:23 -0000 On 16/05/2022 17:10, Tamar Christina via Binutils wrote: > Hi All, > > In Armv9-A SVE is mandatory, and for SVE FP16 is mandatory. This fixes a disconnect > between GCC and binutils where GCC has FP16 on by default and gas doesn't. > > build on native hardware and regtested on aarch64-none-linux-gnu and no issues. > > Ok for master? and for backport to binutils-2.38? > OK for master. I think you'll need Nick to approve a backport, though. I would recommend that. R. > Thanks, > Tamar > > include/ChangeLog: > > 2022-05-16 Tamar Christina > > * opcode/aarch64.h (AARCH64_ARCH_V9_FEATURES): Add AARCH64_FEATURE_F16. > > --- inline copy of patch -- > diff --git a/include/opcode/aarch64.h b/include/opcode/aarch64.h > index a4912930c8f1e564bec6ec46a649ac1205093bd1..8fe27900cba3e897880a3966cc2a17e89dbdf018 100644 > --- a/include/opcode/aarch64.h > +++ b/include/opcode/aarch64.h > @@ -140,6 +140,7 @@ typedef uint32_t aarch64_insn; > | AARCH64_FEATURE_HBC) > > #define AARCH64_ARCH_V9_FEATURES (AARCH64_FEATURE_V9 \ > + | AARCH64_FEATURE_F16 \ > | AARCH64_FEATURE_SVE \ > | AARCH64_FEATURE_SVE2) > #define AARCH64_ARCH_V9_1_FEATURES (AARCH64_ARCH_V8_6_FEATURES) > > > >