[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: allocsa module update
From: |
Ralf Wildenhues |
Subject: |
Re: allocsa module update |
Date: |
Tue, 8 Aug 2006 15:27:08 +0200 |
User-agent: |
Mutt/1.5.11 |
Hello Bruno,
* Bruno Haible wrote on Tue, Aug 08, 2006 at 03:09:12PM CEST:
> Ralf Wildenhues wrote:
> > > + #elif defined _AIX
> > > + /* Work around an AIX 3.2.5 xlc bug with enums constants defined as
> > > offsetof
> > > + values. */
> > > + # define sa_alignof(type) 4
> >
> > It looks rather scary to me, ...
> > But second, newer AIX versions may align 'long long' on an 8 byte
> > boundary. So, if sa_alignof needs only mention an upper bound (which I
> > don't know, as I don't know this code), then that should be 8.
>
> You're right; I didn't think at newer versions of AIX. I'm applying the
> appended patch.
Thanks. Note this still doesn't cope with -qalign, nor would I bet
money that AIX won't ever use 8 byte alignment for 4 byte types (or
similar).
> > But maybe it's either not worth catering for AIX 3.2.5 any more (hey, AIX
> > 4.3.3 has reached end of service life for a while)
>
> I cared about it as long as an AIX 3.2.5 machine was part of my zoo of
> machine. Maybe Nelson Beebe still has one in his zoo.
Fine with me.
> > What was the original bug report that prompted this, by the way? It
> > should affect some Autoconf proper code as well.
>
> Noticed it while building gettext or libiconv. I didn't bother to report
> a bug to myself :-)
Sure. But since we (well, I ;-) still don't know what the bug is, and I
don't have access to AIX 3.2.5 to check, I can't really judge whether...
> - I don't see any autoconf code use the equivalent of __alignof__.
...AC_CHECK_ALIGNOF needs to be adjusted for this.
Cheers,
Ralf