Re: [ros-dev] [ros-diffs] [akhaldi] 59598: [CMAKE/GCC] * Compile debug builds using -O1 instead of -Os. This results in faster compilation time with no measurable difference in runtime performance according to tests. CORE-7...

[Timo Kreuzer]

It used to be Os (optimize for size). not O2.
We could use O2 for the release builds IMO.

Timo

Am 31.07.2013 16:39, schrieb Alex Ionescu:
>   I don't really like this. Maybe only do this on
> the build server, or
>  at least don't do it to kernel code. "Measurable" difference in
>  performance doesn't really mean much on an OS that does not have any
>  performance infrastructure, but trust me, there's a difference between
>  O1 and O2.
>
>  Best regards,
>  Alex Ionescu
>
>
>  On Wed, Jul 31, 2013 at 6:04 AM, <akhaldi(a)svn.reactos.org
>  <mailto:[e-mail]>> wrote:
>
>      Author: akhaldi
>      Date: Wed Jul 31 13:04:38 2013
>      New Revision: 59598
>
>      URL: http://svn.reactos.org/svn/reactos?rev=59598&view=rev
>      Log:
>      [CMAKE/GCC]
>      * Compile debug builds using -O1 instead of -Os. This results in
>      faster compilation time with no measurable difference in runtime
>      performance according to tests.
>      CORE-7374 #resolve #comment Committed in r59598.
>
>      Modified:
>          trunk/reactos/cmake/config.cmake
>
>      Modified: trunk/reactos/cmake/config.cmake
>      URL:
>     
> http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/config.cmake?rev=595…
>      ==============================================================================
>      --- trunk/reactos/cmake/config.cmake    [iso-8859-1] (original)
>      +++ trunk/reactos/cmake/config.cmake    [iso-8859-1] Wed Jul 31
>      13:04:38 2013
>      @@ -10,7 +10,7 @@
>       set(TUNE "i686" CACHE STRING
>       "Which CPU ReactOS should be optimized for.")
>
>      -set(OPTIMIZE "1" CACHE STRING
>      +set(OPTIMIZE "4" CACHE STRING
>       "What level of optimization to use.
>        0 = off
>        1 = Default option, optimize for size (-Os) with some additional
>      options
>
>
>
>
>
>  _______________________________________________
>  Ros-dev mailing list
>  Ros-dev(a)reactos.org
>  http://www.reactos.org/mailman/listinfo/ros-dev