From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18354 invoked by alias); 13 Aug 2014 22:22:33 -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 18336 invoked by uid 89); 13 Aug 2014 22:22:33 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.7 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mailout11.t-online.de Received: from mailout11.t-online.de (HELO mailout11.t-online.de) (194.25.134.85) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Wed, 13 Aug 2014 22:22:31 +0000 Received: from fwd32.aul.t-online.de (fwd32.aul.t-online.de [172.20.26.144]) by mailout11.t-online.de (Postfix) with SMTP id 592CA15BDAA; Thu, 14 Aug 2014 00:22:26 +0200 (CEST) Received: from [192.168.0.103] (Z4C-BvZfrhPHVIdb6QbHQB9wRBe3sTUmsa4Ngf3H1qngOdzBQW4wdZLzVhD8gCbwMC@[84.175.194.76]) by fwd32.t-online.de with (TLSv1:ECDHE-RSA-AES256-SHA encrypted) esmtp id 1XHgw5-1zoPIW0; Thu, 14 Aug 2014 00:22:25 +0200 Message-ID: <1407968544.10959.6.camel@yam-132-YW-E178-FTW> Subject: Re: __intN patch 3/5: main __int128 -> __intN conversion. From: Oleg Endo To: DJ Delorie Cc: gcc-patches@gcc.gnu.org Date: Wed, 13 Aug 2014 22:22:00 -0000 In-Reply-To: <201408132211.s7DMBGBu016387@greed.delorie.com> References: <201408132211.s7DMBGBu016387@greed.delorie.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Mime-Version: 1.0 X-IsSubscribed: yes X-SW-Source: 2014-08/txt/msg01400.txt.bz2 On Wed, 2014-08-13 at 18:11 -0400, DJ Delorie wrote: > This is the main __int128 -> __intN conversion patch. We support up > to four __int types per target, which may include __int128. I > include the minimal change to the msp430 backend to illustrate how to > add a target-specific __int type. > > [...] > Index: gcc/machmode.h > =================================================================== > --- gcc/machmode.h (revision 213886) > +++ gcc/machmode.h (working copy) > @@ -342,7 +342,19 @@ extern void init_adjust_machine_modes (v > GET_MODE_PRECISION (MODE2)) > > #define HWI_COMPUTABLE_MODE_P(MODE) \ > (SCALAR_INT_MODE_P (MODE) \ > && GET_MODE_PRECISION (MODE) <= HOST_BITS_PER_WIDE_INT) > > +typedef struct { > + /* These parts are initailized by genmodes output */ > + unsigned int bitsize; > + enum machine_mode m; > + /* RID_* is RID_INTN_BASE + index into this array */ > +} int_n_data_t; > + A while ago I've removed a couple of those 'typedef struct' things, as they are not required in C++ anymore. Is there any particular reason why this couldn't be simply 'struct int_n_data_t' ? Cheers, Oleg