From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 102296 invoked by alias); 21 Jul 2017 21:15:48 -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 101639 invoked by uid 89); 21 Jul 2017 21:15:40 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=offline, love, friends X-HELO: gate.crashing.org Received: from gate.crashing.org (HELO gate.crashing.org) (63.228.1.57) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 21 Jul 2017 21:15:38 +0000 Received: from gate.crashing.org (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.13.8) with ESMTP id v6LLFIQh000383; Fri, 21 Jul 2017 16:15:20 -0500 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id v6LLFEbn000379; Fri, 21 Jul 2017 16:15:14 -0500 Date: Fri, 21 Jul 2017 21:15:00 -0000 From: Segher Boessenkool To: Carl Love Cc: gcc-patches@gcc.gnu.org, David Edelsohn , Bill Schmidt Subject: Re: [PATCH, rs6000] vec_mule and vec_mulo builtin fix Message-ID: <20170721211512.GI13471@gate.crashing.org> References: <1500591361.19855.44.camel@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1500591361.19855.44.camel@us.ibm.com> User-Agent: Mutt/1.4.2.3i X-IsSubscribed: yes X-SW-Source: 2017-07/txt/msg01352.txt.bz2 On Thu, Jul 20, 2017 at 03:56:01PM -0700, Carl Love wrote: > The following patch is a reworked patch to fix the bugs in the vec_mule > and vec_mulo patch that had to be reverted. The reverted fix, added the > correct mule and mulo instructions with vectorization. The > vectorization support resulted in vectorization in several testcases to > fail. This patch adds the correct instructions for the vec_mule and > vec_mulo builtins without vectorization. The goal is add the > vectorization support again in a future patch. Ah, as you explained offline, you removed vec_widen_umult_even_v4si and friends. Okay :-) The patch is okay for trunk then. Thanks! Segher