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 E5250385842E for ; Thu, 16 Dec 2021 10:45:33 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org E5250385842E 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 6C4EE1474; Thu, 16 Dec 2021 02:45:33 -0800 (PST) Received: from localhost (e121540-lin.manchester.arm.com [10.32.98.88]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 78EEA3F774; Thu, 16 Dec 2021 02:45:32 -0800 (PST) From: Richard Sandiford To: Przemyslaw Wirkus Mail-Followup-To: Przemyslaw Wirkus , "gcc-patches\@gcc.gnu.org" , "christophe.lyon\@linaro.org" , Richard Earnshaw , Kyrylo Tkachov , Marcus Shawcroft , richard.sandiford@arm.com Cc: "gcc-patches\@gcc.gnu.org" , "christophe.lyon\@linaro.org" , Richard Earnshaw , Kyrylo Tkachov , Marcus Shawcroft Subject: Re: [PATCH][GCC] aarch64: fix: ls64 tests fail on aarch64_be [PR103729] References: Date: Thu, 16 Dec 2021 10:45:31 +0000 In-Reply-To: (Przemyslaw Wirkus's message of "Thu, 16 Dec 2021 10:36:21 +0000") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-12.4 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org 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: Thu, 16 Dec 2021 10:45:35 -0000 Przemyslaw Wirkus writes: > Hi, > > This patch is sorting issue with LS64 intrinsics tests failing with > AArch64_be targets. > > Regtested on aarch64_be_elf and aarch64_elf, no issues. > > OK to install? > > gcc/ChangeLog: > > PR target/PR103729 > * config/aarch64/aarch64-simd.md (aarch64_movv8di): Allow big endian > targets to move V8DI. OK, thanks. Richard > diff --git a/gcc/config/aarch64/aarch64-simd.md b/gcc/config/aarch64/aarch64-simd.md > index 9ebf795a624f0183e0333349d0db7a71ba2d17dd..f95a7e1d91c97c9e981d75e71f0b49c02ef748ba 100644 > --- a/gcc/config/aarch64/aarch64-simd.md > +++ b/gcc/config/aarch64/aarch64-simd.md > @@ -7265,9 +7265,8 @@ (define_insn "*aarch64_mov" > (define_insn "*aarch64_movv8di" > [(set (match_operand:V8DI 0 "nonimmediate_operand" "=r,m,r") > (match_operand:V8DI 1 "general_operand" " r,r,m"))] > - "!BYTES_BIG_ENDIAN > - && (register_operand (operands[0], V8DImode) > - || register_operand (operands[1], V8DImode))" > + "(register_operand (operands[0], V8DImode) > + || register_operand (operands[1], V8DImode))" > "#" > [(set_attr "type" "multiple,multiple,multiple") > (set_attr "length" "32,16,16")]