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...

[Alex Ionescu]

We should use -ofast for release builds, imo :)

Best regards,
Alex Ionescu


On Wed, Jul 31, 2013 at 8:40 AM, Alex Ionescu <ionucu(a)videotron.ca> wrote:

>   O2 is Os :)
>
>  Best regards,
>  Alex Ionescu
>
>
>  On Wed, Jul 31, 2013 at 8:04 AM, Timo Kreuzer <timo.kreuzer(a)web.de> wrote:
>
> >  
> >  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> 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 [e-mail]://www.reactos.org/mailman/listinfo/ros-dev
> >
> >
> >
> >  _______________________________________________
> >  Ros-dev mailing list
> >  Ros-dev(a)reactos.org
> >  http://www.reactos.org/mailman/listinfo/ros-dev
> >
> >  
>