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