cvs commit: src/share/mk sys.mk

Bruce Evans brde at optusnet.com.au
Wed Jun 27 19:42:04 UTC 2007


On Wed, 27 Jun 2007, Xin LI wrote:

> Andrey Chernov wrote:
>> I just sent this cross-building compatible variant to re@ for approve:
>>
>> --- sys.mk.orig	2007-06-27 14:04:15.000000000 +0400
>> +++ sys.mk	2007-06-27 14:04:33.000000000 +0400
>> @@ -35,7 +35,7 @@
>>  CFLAGS		?=	-O
>>  .else
>>  CC		?=	cc
>> -CFLAGS		?=	-O2 -fno-strict-aliasing -fno-tree-vrp -pipe
>> +CFLAGS		?=	-O1 -fno-strict-aliasing -pipe
>>  .endif
>
> Nitpicking: I think -O1 implies no strict-aliasing.  So -O1 -pipe might
> be just Ok.

Still has a style bug: -O implies -O.  If changing it, back out the commit
that changed it from -O.  I still use -O in most cases since -O2 is just a
waste of time in most cases.

Bruce


More information about the cvs-src mailing list