[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