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 A7DFA3858D37 for ; Mon, 18 Mar 2024 14:37:11 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org A7DFA3858D37 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 ARC-Filter: OpenARC Filter v1.0.0 sourceware.org A7DFA3858D37 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=63.228.1.57 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1710772633; cv=none; b=TwZOFLZIDOeZ6dQuyp/s83nyCyzRmbPZHtN/U8s28k4OREyeWcHa7mjl/8zeNJbSbon5u8Od+tzj4snEVFpv/+xBYhTqjO5qej9tvRpU0hv3DP4s5znR3sDS93st8OPpmY9coJ7u7XdfudGSAvDPsnk/KeXxHtFWCWSZd9+vmjA= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1710772633; c=relaxed/simple; bh=j0VsllGkkLakJQnt+0L7Rh2IlBHxaUzj3M2dTkzh8cc=; h=Date:From:To:Subject:Message-ID:Mime-Version; b=GkeMLlSuqpkk1On9Z0QLb8QoFLV1A/YKyU2WDptDPOnZIq26XOcIwCLltvTLhu5i/kcrFqKx2tzq3F564HPUnK9bCI02EJjQMSplNuhRfUW0fJphE5TJYt4U/osHVdDXy9qrkmshOsM+yuLFUgIfF7WsKSLs1H7sXRHiFlbt26E= ARC-Authentication-Results: i=1; server2.sourceware.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 42IEaBhq010801; Mon, 18 Mar 2024 09:36:11 -0500 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id 42IEa9T1010800; Mon, 18 Mar 2024 09:36:09 -0500 X-Authentication-Warning: gate.crashing.org: segher set sender to segher@kernel.crashing.org using -f Date: Mon, 18 Mar 2024 09:36:09 -0500 From: Segher Boessenkool To: jeevitha Cc: GCC Patches , "Kewen.Lin" , Peter Bergner Subject: Re: [PATCH] rs6000: Fix issue in specifying PTImode as an attribute [PR106895] Message-ID: <20240318143609.GX19790@gate.crashing.org> References: <9ea38d87-180f-46b5-a723-45061680980f@linux.vnet.ibm.com> <92507c51-9ff0-4e67-8a6c-7ffb58bc8537@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <92507c51-9ff0-4e67-8a6c-7ffb58bc8537@linux.vnet.ibm.com> User-Agent: Mutt/1.4.2.3i X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,JMQ_SPF_NEUTRAL,KAM_DMARC_STATUS,SPF_HELO_PASS,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE 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 Fri, Feb 23, 2024 at 03:04:13PM +0530, jeevitha wrote: > PTImode attribute assists in generating even/odd register pairs on 128 bits. It is a mode, not an attribute. Attributes are on declarations, while modes are on a much more fundamental level (every value has a mode, in GCC!) > When the user specifies PTImode as an attribute, it breaks because there is no > internal type to handle this mode . We have created a tree node with dummy type > to handle PTImode. You are talking about the mode attribute. Aha. This attribute says the mode of the datum is something specific; it is the only way a user can specify the mode directly. Not something you want to use normally, but it's a nice escape hatch (like here). > We are not documenting this dummy type since users are not > allowed to use this type externally. Not sure what this is meant to mean? What does "allowed to" mean, even? We do not forbid people from doing anything (we can discourage them though). Or dso you mean something just doesn't work? > gcc/ > PR target/106895 > * config/rs6000/rs6000.h (enum rs6000_builtin_type_index): Add fields > to hold PTImode type. An enum does not have fields. What do you actually mean? > --- a/gcc/config/rs6000/rs6000.h > +++ b/gcc/config/rs6000/rs6000.h > @@ -2304,6 +2304,7 @@ enum rs6000_builtin_type_index > RS6000_BTI_ptr_vector_quad, > RS6000_BTI_ptr_long_long, > RS6000_BTI_ptr_long_long_unsigned, > + RS6000_BTI_PTI, Please call it RS6000_BTI_INTPTI, to be more in line with the naming of other things. With that fixed it should be good. Please repost with a good commit comment and changelog :-) Thanks, Segher