This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: __intN patch 3/5: main __int128 -> __intN conversion.
- From: Oleg Endo <oleg dot endo at t-online dot de>
- To: DJ Delorie <dj at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 14 Aug 2014 00:22:24 +0200
- Subject: Re: __intN patch 3/5: main __int128 -> __intN conversion.
- Authentication-results: sourceware.org; auth=none
- References: <201408132211 dot s7DMBGBu016387 at greed dot delorie dot com>
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<N> 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<N> 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