[Libreoffice-commits] core.git: solenv/gbuild

[Tor Lillqvist]

 solenv/gbuild/platform/com_MSC_defs.mk |   11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

New commits:
commit 81d657f351650b17b05704c65abc2863473761dc
Author: Tor Lillqvist <tml at collabora.com>
Date:   Tue Oct 1 20:37:13 2013 +0300

    Try again to use -O2 but keep -Oy- this time
    
    Change-Id: Iee396f289df4dc4134c30e6b6b1126d2745c1f6f

diff --git a/solenv/gbuild/platform/com_MSC_defs.mk b/solenv/gbuild/platform/com_MSC_defs.mk
index e4ff2bb..b57dad0 100644
--- a/solenv/gbuild/platform/com_MSC_defs.mk
+++ b/solenv/gbuild/platform/com_MSC_defs.mk
@@ -275,16 +275,7 @@ gb_CFLAGS+=$(gb_DEBUG_CFLAGS)
 gb_CXXFLAGS+=$(gb_DEBUG_CFLAGS)
 endif
 
-# The optimisation options here might seem like micro-management (see
-# http://www.ffuts.org/blog/dont-spend-time-micromanaging-optimization-options-in-visual-c/
-# ), and the exact rationale for this selection of options for
-# gb_COMPILEROPTFLAGS is probably lost in history. We used -Ob1 -Oxs
-# -Oy- already in OOo 3.2 at least.
-
-# Anyway, changing it to just -O2 broke unit tests. So there must be
-# some reason for this particular choice. And if you want to change it
-# to something simpler, please run a full make check;)
-gb_COMPILEROPTFLAGS := -Ob1 -Oxs -Oy-
+gb_COMPILEROPTFLAGS := -O2 -Oy-
 gb_COMPILERNOOPTFLAGS := -Od
 
 ifeq ($(gb_FULLDEPS),$(true))