From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5194 invoked by alias); 6 Nov 2013 02:42:33 -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 5185 invoked by uid 89); 6 Nov 2013 02:42:32 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.4 required=5.0 tests=AWL,BAYES_50,FREEMAIL_FROM,RDNS_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no version=3.3.2 X-HELO: mail-vb0-f47.google.com Received: from Unknown (HELO mail-vb0-f47.google.com) (209.85.212.47) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Wed, 06 Nov 2013 02:40:39 +0000 Received: by mail-vb0-f47.google.com with SMTP id m10so2946642vbh.6 for ; Tue, 05 Nov 2013 18:40:31 -0800 (PST) MIME-Version: 1.0 X-Received: by 10.52.35.20 with SMTP id d20mr370009vdj.33.1383705631258; Tue, 05 Nov 2013 18:40:31 -0800 (PST) Received: by 10.221.63.73 with HTTP; Tue, 5 Nov 2013 18:40:31 -0800 (PST) In-Reply-To: <1383611317.6275.332.camel@gnopaine> References: <1383542887.6275.299.camel@gnopaine> <1383611317.6275.332.camel@gnopaine> Date: Wed, 06 Nov 2013 02:45:00 -0000 Message-ID: Subject: Re: [PATCH, rs6000] (1/3) Reverse meanings of multiply even/odd for little endian From: David Edelsohn To: Bill Schmidt Cc: GCC Patches Content-Type: text/plain; charset=ISO-8859-1 X-SW-Source: 2013-11/txt/msg00501.txt.bz2 On Mon, Nov 4, 2013 at 7:28 PM, Bill Schmidt wrote: > Hi, > > Here's a new version of this patch, revised according to Richard > Sandiford's suggestions. Unfortunately the diffing is a little bit ugly > for this version. > > Bootstrapped and tested on powerpc64{,le}-unknown-linux-gnu with no > regressions. Is this ok for trunk? > > Thanks, > Bill > > > 2013-11-04 Bill Schmidt > > * config/rs6000/altivec.md (vec_widen_umult_even_v16qi): Change > define_insn to define_expand that uses even patterns for big > endian and odd patterns for little endian. > (vec_widen_smult_even_v16qi): Likewise. > (vec_widen_umult_even_v8hi): Likewise. > (vec_widen_smult_even_v8hi): Likewise. > (vec_widen_umult_odd_v16qi): Likewise. > (vec_widen_smult_odd_v16qi): Likewise. > (vec_widen_umult_odd_v8hi): Likewise. > (vec_widen_smult_odd_v8hi): Likewise. > (altivec_vmuleub): New define_insn. > (altivec_vmuloub): Likewise. > (altivec_vmulesb): Likewise. > (altivec_vmulosb): Likewise. > (altivec_vmuleuh): Likewise. > (altivec_vmulouh): Likewise. > (altivec_vmulesh): Likewise. > (altivec_vmulosh): Likewise. Okay. Unfortunately there is no way to avoid an ugly solution. Thanks, David