bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: new module 'stdalign'


From: Paul Eggert
Subject: Re: new module 'stdalign'
Date: Tue, 12 Jul 2011 10:23:12 -0700
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.18) Gecko/20110621 Fedora/3.1.11-1.fc14 Thunderbird/3.1.11

Thanks for looking into that.  Some thoughts.

This module will have problems with the gnulib model,
since it "feels" like a *.in.h module, and yet it
can't be written to support arbitrary compilers like
the other *.in.h modules can.  So it won't be suitable
for applications that need to install headers in arbitrary
locations (as stdint.in.h is).

Some compilers other than GCC support __aligned__ and __alignof__.
Oracle Solaris Studio is one that I know about.

I'd feel more comfortable if the module checked for
an existing stdalign.h and did not replace one that
already exists.  This will be more likely to do the
right thing for software distributions that we are
generating now but that will be compiled on non-GCC
platforms in 2012.

Given the above, it sounds like it'd be better for 'configure' to test
for the presence of the __aligned__ and __alignof__ keywords.
That is what GCC's configure.in does, for __aligned__,
and that way I could modify Emacs to use the new stdalign module.
That would simplify gnulib's maintenance task too, as we wouldn't
need to keep track of which compilers support these keywords.

Should stdalign use the existing alignof module
rather than copying its code, a bit differently?
Any namespace pollution issues should be easily resolvable,
I'd think.

Would you mind changing the list of maintainers to "all",
or at least to include Jim and me?




reply via email to

[Prev in Thread] Current Thread [Next in Thread]