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 51B413858435 for ; Mon, 7 Mar 2022 14:17:03 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 51B413858435 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 EA5DA1042; Mon, 7 Mar 2022 06:17:02 -0800 (PST) Received: from [10.57.6.57] (unknown [10.57.6.57]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 795ED3F66F; Mon, 7 Mar 2022 06:17:02 -0800 (PST) Message-ID: Date: Mon, 7 Mar 2022 14:16:31 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.6.1 Subject: Re: [arm] MVE: Relax addressing modes for full loads and stores Content-Language: en-US To: kyrylo Tkachov Cc: GCC Patches References: <20220113145645.4077141-1-christophe.lyon@foss.st.com> <20220113145645.4077141-16-christophe.lyon@foss.st.com> <52549bc7-2784-c721-0420-67ad4d40a5ca@arm.com> From: "Andre Vieira (lists)" In-Reply-To: X-Spam-Status: No, score=-5.3 required=5.0 tests=BAYES_00, BODY_8BITS, HTML_MESSAGE, KAM_DMARC_STATUS, NICE_REPLY_A, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 07 Mar 2022 14:17:04 -0000 On 17/01/2022 07:48, Christophe Lyon wrote: > Hi André, > > On Fri, Jan 14, 2022 at 6:03 PM Andre Vieira (lists) via Gcc-patches > wrote: > > Hi Christophe, > > This patch relaxes the addressing modes for the mve full load and > stores > (by full loads and stores I mean non-widening or narrowing loads and > stores resp). The code before was requiring a LO_REGNUM for these, > where > this is only a requirement if the load is widening or the store > narrowing. > > So with this your patch should not be necessary. > > Regression tested on arm-none-eabi-gcc.  Can you please confirm this > fixes the issue you were seeing too? > > > Yes, I confirm this fixes the problem I was fixing with my patch #15 > in my MVE/VCMP/VCOND series. > I'll drop it. > > Thanks! > > Christophe > > > gcc/ChangeLog: > >          * config/arm/arm.h (MVE_STN_LDW_MODE): New MACRO. >          * config/arm/arm.c (mve_vector_mem_operand): Relax > constraint on >          base register for non widening loads or narrowing stores. > > > Kind Regards, > Andre Vieira > Ping, I noticed this also fixes PR 104790. Kind regards, Andre