From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11024 invoked by alias); 27 Mar 2012 17:10:24 -0000 Received: (qmail 11010 invoked by uid 22791); 27 Mar 2012 17:10:21 -0000 X-SWARE-Spam-Status: No, hits=-2.9 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from localhost (HELO gcc.gnu.org) (127.0.0.1) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 27 Mar 2012 17:10:08 +0000 From: "jakub at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/52607] v4df __builtin_shuffle with {0,2,1,3} or {1,3,0,2} Date: Tue, 27 Mar 2012 17:25:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: target X-Bugzilla-Keywords: X-Bugzilla-Severity: enhancement X-Bugzilla-Who: jakub at gcc dot gnu.org X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2012-03/txt/msg02352.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52607 --- Comment #20 from Jakub Jelinek 2012-03-27 17:10:00 UTC --- Thanks for working on this. I don't like much the calls to ix86_expand_vec_perm_const_1, if you are looking for exactly two insn permutations, then really the two insn permutation functions should be groupped together into expand_vec_perm_2 and you should call that instead, or if it is 1 or 2, then expand_vec_perm_1 || expand_vec_perm_2. expand_vec_perm_vperm2f128_merge has probably swapped the meaning of dfirst and dsecond permutations when it first performs the dsecond permutation. Lastly for each routine it is desirable to think whether it might be useful for other vector modes (likely 32-byte only) for TARGET_AVX2.