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

Gaetan Nadon memsize at videotron.ca
Tue Feb 23 12:55:27 PST 2010


On Tue, 2010-02-23 at 11:11 -0800, Jeremy Huddleston wrote:

> On Feb 23, 2010, at 11:07, Keith Packard wrote:
> 
> > On Tue, 23 Feb 2010 13:37:18 -0500, Gaetan Nadon  
> > <memsize at videotron.ca> wrote:
> >
> >> This patch will ensure the xserver continues to suppress the
> >> optimization, based on strict aliasing rules, after the option
> >> is removed from $CWARNFLAGS. There is no change in the object
> >> code produced.
> >
> > I don't think we need to allow any of the code in the X server to be
> > 'optimized' in this fashion, so I don't see a need to allow for
> > per-directory selection.
> 
> It would be helpful if I had the option to remove this flag from  
> XQuartz.
> 
> 

Patch #1 is designed to maintain status-quo following the removal of
-fno-strict-aliasing from CWARNFLAGS and does not require subsequent
patches. It's a 'global' setting.

Patch #2 is designed to maintain status-quo, but making it easy for
various parts of the code to enable optimization if appropriate. It was
later altered for XQuartz, which could have been done through a
subsequent patch.

The alternative to patch #2 is for components to append
-fstrict-aliasing to override -fno-strict-aliasing. 


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100223/893829cb/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100223/893829cb/attachment.pgp>


More information about the xorg-devel mailing list