From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25431 invoked by alias); 1 Oct 2014 10:35:25 -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 25414 invoked by uid 89); 1 Oct 2014 10:35:24 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Wed, 01 Oct 2014 10:35:22 +0000 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s91AZKXK024475 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 1 Oct 2014 06:35:21 -0400 Received: from tucnak.zalov.cz (ovpn-116-116.ams2.redhat.com [10.36.116.116]) by int-mx14.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id s91AZI0d002301 (version=TLSv1/SSLv3 cipher=AES128-GCM-SHA256 bits=128 verify=NO); Wed, 1 Oct 2014 06:35:20 -0400 Received: from tucnak.zalov.cz (localhost [127.0.0.1]) by tucnak.zalov.cz (8.14.9/8.14.9) with ESMTP id s91AZGre031257; Wed, 1 Oct 2014 12:35:17 +0200 Received: (from jakub@localhost) by tucnak.zalov.cz (8.14.9/8.14.9/Submit) id s91AZEAr031256; Wed, 1 Oct 2014 12:35:14 +0200 Date: Wed, 01 Oct 2014 10:35:00 -0000 From: Jakub Jelinek To: Uros Bizjak Cc: Evgeny Stupachenko , "H.J. Lu" , Richard Henderson , GCC Patches , Richard Biener Subject: Re: [PATCH 1/2, x86] Add palignr support for AVX2. Message-ID: <20141001103514.GO1986@tucnak.redhat.com> Reply-To: Jakub Jelinek References: <53FCB5ED.7040209@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) X-IsSubscribed: yes X-SW-Source: 2014-10/txt/msg00027.txt.bz2 On Wed, Oct 01, 2014 at 12:28:51PM +0200, Uros Bizjak wrote: > On Wed, Oct 1, 2014 at 12:16 PM, Evgeny Stupachenko wrote: > > Getting back to initial patch, is it ok? > > IMO, we should start with Jakub's proposed patch [1] > > [1] https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00010.html That doesn't compile, will post a new version; got interrupted when I found that in GCC_TEST_RUN_EXPENSIVE=1 make check-gcc RUNTESTFLAGS='--target_board=unix/-mavx2 dg-torture.exp=vshuf*.c' one test is miscompiled even with unpatched compiler, debugging that now. That said, my patch will not do anything about the case Mark mentioned { 1, 2, 3, ..., 31, 0 } permutation, for that we can't do vpalignr followed by vpshufb or similar, but need to do some permutation first and then vpalignr on the result. So it would need a new routine. It is still a 2 insn permutation, not 6, and needs different algorithm, so sharing the same routine for that is undesirable. Jakub