This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v2 1/2] Y2038: Add 64-bit time for all architectures
- From: Ben Hutchings <ben dot hutchings at codethink dot co dot uk>
- To: "Albert ARIBAUD (3ADEV)" <albert dot aribaud at 3adev dot fr>, libc-alpha at sourceware dot org
- Date: Mon, 30 Jul 2018 13:41:04 +0800
- Subject: Re: [PATCH v2 1/2] Y2038: Add 64-bit time for all architectures
- References: <20180614135116.8767-1-albert.aribaud@3adev.fr> <20180614135116.8767-2-albert.aribaud@3adev.fr> <1532929095.21552.67.camel@codethink.co.uk>
On Mon, 2018-07-30 at 13:38 +0800, Ben Hutchings wrote:
> On Thu, 2018-06-14 at 15:51 +0200, Albert ARIBAUD (3ADEV) wrote:
> > * Add macro __TIMESIZE equal to the bit size of time_t.
> > It equals the architecture __WORDSIZE except for x32
> > where it equals 64.
> >
> > * Add type __time64_t which is always 64-bit. On 64-bit
> > architectures and on x32, it is #defined as time_t.
> > On other architectures, it has its own type.
>
> [...]
> > --- /dev/null
> > +++ b/sysdeps/unix/sysv/linux/x86/bits/timesizes.h
>
> [...]
> > +/* Both x86-64 and x32 use 64-bit time. */
> > +#define __TIME64_T_TYPE __TIME_T_TYPE
>
> [...]
>
> But won't this file be used for regular 32-bit x86 (i386/i686) as well?
Never mind, I see this got fixed in v5.
Ben.
--
Ben Hutchings, Software Developer Codethink Ltd
https://www.codethink.co.uk/ Dale House, 35 Dale Street
Manchester, M1 2HF, United Kingdom