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, <akha...@svn.reactos.org
> <mailto:akha...@svn.reactos.org>> 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=59598&r1=59597&r2=59598&view=diff
>     
> ==============================================================================
>     --- 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@reactos.org
> http://www.reactos.org/mailman/listinfo/ros-dev

_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev

Reply via email to