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 66A083858421 for ; Tue, 20 Dec 2022 18:04:25 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 66A083858421 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 2BKI3OvZ025661; Tue, 20 Dec 2022 12:03:24 -0600 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id 2BKI3NC9025660; Tue, 20 Dec 2022 12:03:23 -0600 X-Authentication-Warning: gate.crashing.org: segher set sender to segher@kernel.crashing.org using -f Date: Tue, 20 Dec 2022 12:03:23 -0600 From: Segher Boessenkool To: HAO CHEN GUI Cc: gcc-patches , David , "Kewen.Lin" , Peter Bergner Subject: Re: [PATCH v6, rs6000] Change mode and insn condition for VSX scalar extract/insert instructions Message-ID: <20221220180323.GS25951@gate.crashing.org> References: <3d530867-c6a2-15bf-fd65-54313622acda@linux.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3d530867-c6a2-15bf-fd65-54313622acda@linux.ibm.com> User-Agent: Mutt/1.4.2.3i X-Spam-Status: No, score=-3.0 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: Hi! On Mon, Dec 19, 2022 at 02:27:57PM +0800, HAO CHEN GUI wrote: > This patch fixes several problems: > 1. The exponent of double-precision can be put into a SImode register. > So "xsxexpdp" doesn't require 64-bit environment. Also "xsxsigdp", > "xsiexpdp" and "xsiexpdpf" can put exponent into a GPR register. > > 2. "TARGET_64BIT" check in insn conditions should be replaced with > "TARGET_POWERPC64" check. > > 3. "lp64" check in test cases should be replaced with "has_arch_ppc64" > check. "ilp32" check should be replaced with "dg-skip-if has_arch_ppc64". These things are independent. Please do independent patches (a series is fine and even preferred, of course). Not only is this much harder to review the way it is, it was harder to write as well, write changelog for, etc. Often when you prepare patches it becomes apparent that you should have structured things a bit differently. That is a good time to do exactly so :-) > This patch keeps outer interfaces of these builtins unchanged. Of course. This patch doesn't edit any builtins, it changes only the machine description. "Of the corresponding builtins" :-) Segher