From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4506 invoked by alias); 14 Nov 2014 12:25:53 -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 4497 invoked by uid 89); 14 Nov 2014 12:25:52 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_PASS autolearn=ham version=3.3.2 X-HELO: service87.mimecast.com Received: from service87.mimecast.com (HELO service87.mimecast.com) (91.220.42.44) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 14 Nov 2014 12:25:49 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.21]) by service87.mimecast.com; Fri, 14 Nov 2014 12:25:46 +0000 Received: from [10.1.209.51] ([10.1.255.212]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Fri, 14 Nov 2014 12:25:46 +0000 Message-ID: <5465F4CA.2010606@arm.com> Date: Fri, 14 Nov 2014 12:45:00 -0000 From: Alan Lawrence User-Agent: Thunderbird 2.0.0.24 (X11/20101213) MIME-Version: 1.0 To: Matthew Fortune CC: "gcc-patches@gcc.gnu.org" , Catherine Moore Subject: Re: [PATCH 0/4][Vectorizer] Reductions: replace VEC_RSHIFT_EXPR with VEC_PERM_EXPR References: <54639D83.7090205@arm.com> <6D39441BF12EF246A7ABCE6654B0235320F6ED2F@LEMAIL01.le.imgtec.org> In-Reply-To: <6D39441BF12EF246A7ABCE6654B0235320F6ED2F@LEMAIL01.le.imgtec.org> X-MC-Unique: 114111412254601001 Content-Type: text/plain; charset=WINDOWS-1252; format=flowed Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2014-11/txt/msg01757.txt.bz2 Ah, I didn't realize Loongson was little-endian only. In that case (with mi= d-end=20 reductions-via-shifts changes pushed) I don't think I have actually broken= =20 anything, or at least, no MIPS platform that exists :). However, yes, that would seem a safe bet (and simpler than my linked patch = that=20 provided a BE version too!). Cheers, Alan Matthew Fortune wrote: >> (for MIPS) https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01481.html, >> although I have not been able to test this as there doesn't seem to be >> any working MIPS/Loongson hardware in the Compile Farm; >=20 > I will post a patch to remove vec_shl and only support vec_shr for > little endian. This is on the basis that loongson only supports > little endian anyway. >=20 > I believe this is a safe thing to do regardless of whether your change > is in place or not. >=20 > Matthew >=20