[PATCH xserver 1/3] config: add -fno-strict-aliasing now that it is out of $CWARNFLAGS

Keith Packard keithp at keithp.com
Thu Feb 25 10:57:10 PST 2010


On Thu, 25 Feb 2010 13:09:55 -0500, Gaetan Nadon <memsize at videotron.ca> wrote:

> Can we assume that all modules who did not have -fno-strict-aliasing
> before Jan 2009 have been working correctly and do not need it? That
> gave about 4 years to detect any problems. I am not sure what's the set
> of modules you have in mind that should have -fno-strict-aliasing.

My main concern is in code dealing with either the wire protocol or
image data, with a minor concern for the rest of the X server which does
DrawablePtr/PixmapPtr/WindowPtr hacks. The protocol code could
presumably be fixed to use void * everywhere, which (iirc) bypasses the
aliasing optimization.

Having this be consistent across all protocol libraries and server
modules seems like a good idea, even if there aren't obvious bugs
present now, future changes in GCC may well cause old code to break in
the future and we'll need a hook to make sure that we can apply options
globally across the system.

Another option we may want to consider is -fwrapv which makes signed and
unsigned int/long/char operators operate over the finite field of 2ⁿ
instead of some random set.

Thanks, gcc developers. I'm sure you can read the ANSI C spec in a way
which justifies your optimizations. Please give me back my language.

> I am available to write as many patches as it takes to do the right
> thing.

I wouldn't admit that in public :-)

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100225/94533c10/attachment.pgp>


More information about the xorg-devel mailing list