From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) by sourceware.org (Postfix) with ESMTP id 0CD9638515F3 for ; Wed, 14 Jul 2021 18:39:53 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 0CD9638515F3 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=kernel.crashing.org Authentication-Results: sourceware.org; spf=fail smtp.mailfrom=kernel.crashing.org Received: from gate.crashing.org (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.14.1) with ESMTP id 16EIcr47001103; Wed, 14 Jul 2021 13:38:53 -0500 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id 16EIcqv8001101; Wed, 14 Jul 2021 13:38:52 -0500 X-Authentication-Warning: gate.crashing.org: segher set sender to segher@kernel.crashing.org using -f Date: Wed, 14 Jul 2021 13:38:52 -0500 From: Segher Boessenkool To: "Kewen.Lin" Cc: GCC Patches , Bill Schmidt , David Edelsohn Subject: Re: [PATCH] rs6000: Support [u]mul3_highpart for vector Message-ID: <20210714183852.GA1583@gate.crashing.org> References: <20210713220712.GU1583@gate.crashing.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.4.2.3i X-Spam-Status: No, score=-6.0 required=5.0 tests=BAYES_00, JMQ_SPF_NEUTRAL, KAM_DMARC_STATUS, TXREP, T_SPF_HELO_PERMERROR, T_SPF_PERMERROR autolearn=no autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Jul 2021 18:39:55 -0000 On Wed, Jul 14, 2021 at 10:12:46AM +0800, Kewen.Lin wrote: > on 2021/7/14 上午6:07, Segher Boessenkool wrote: > > Hi! > > > > On Tue, Jul 13, 2021 at 04:58:42PM +0800, Kewen.Lin wrote: > >> This patch is to make Power10 newly introduced vector > >> multiply high (part) instructions exploited in vectorized > >> loops, it renames existing define_insns as standard pattern > >> names. It depends on that patch which enables vectorizer > >> to recog mul_highpart. > > > > It actually is correct already, it will just not be used yet, right? > > Yes, the names are just not standard. :) I meant after this patch is applied :-) Doesn't change much though -- applying it right now is fine, but you can wait for the generic code to get in first, to make the new tests not fail. Segher