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 607913858D32 for ; Tue, 21 Feb 2023 14:10:29 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 607913858D32 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=kernel.crashing.org Authentication-Results: sourceware.org; spf=pass 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 31LE9SWZ002627; Tue, 21 Feb 2023 08:09:28 -0600 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id 31LE9SHc002619; Tue, 21 Feb 2023 08:09:28 -0600 X-Authentication-Warning: gate.crashing.org: segher set sender to segher@kernel.crashing.org using -f Date: Tue, 21 Feb 2023 08:09:27 -0600 From: Segher Boessenkool To: Ajit Agarwal Cc: gcc-patches , bergner@linux.ibm.com Subject: Re: [PATCH] rs6000: fmr gets used instead of faster xxlor [PR93571] Message-ID: <20230221140927.GQ25951@gate.crashing.org> References: <16fa34b8-ad8a-20f2-b285-3b3f5bf5d5b2@linux.ibm.com> <20230217172319.GL25951@gate.crashing.org> <5f800afe-f162-0d56-3c78-6ee93610e201@linux.ibm.com> <20230221110448.GP25951@gate.crashing.org> <46ec2582-b3a8-52c9-584a-9d282d26fb79@linux.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <46ec2582-b3a8-52c9-584a-9d282d26fb79@linux.ibm.com> User-Agent: Mutt/1.4.2.3i X-Spam-Status: No, score=-2.9 required=5.0 tests=BAYES_00,JMQ_SPF_NEUTRAL,KAM_DMARC_STATUS,SPF_HELO_PASS,SPF_PASS,TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: On Tue, Feb 21, 2023 at 06:00:52PM +0530, Ajit Agarwal wrote: > On 21/02/23 4:34 pm, Segher Boessenkool wrote: > > Please domn't use a switch, it isn't needed. Instead use the "isa" > > attribute (with p7v here), and put the preferred alternative first. > > I am not sure how this is possible without switch and using only "isa". You have the "p7v" "xxlor" alternative earlier than the "*" "fmr" alternative. You can have an "xxlor" for contraints "d", but probably the best (and certainly the easiest) is to just move the existing xxlor to before fmr. Oh, the existing xxlor alternative is implicitly isa p7v, the "wa" constraint causes that. It may be nicer to mark it explicitly p7v as well, nicer for the reader. Btw, please update the other similar patterns at the same time? There are eight patterns with fmr in rs6000.md (the four in dfp.md should probably not be touched); not all are similar so should be in separate patches, if changed at all, but a bunch are completely analogous so should not diverge. (It is fine to first do this one pattern only, until we have worked out all kinks, but all should be committed at the same time). Thanks, Segher