mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add comment about our choice of gb_COMPILEROPTFLAGS

Browse Source 
Change-Id: Iaeb9e1637ab4b85735a44f7cd5428e0e0c47c91d
This commit is contained in:
Tor Lillqvist
2013-09-09 21:12:31 +03:00
parent 788dec27c2
commit 732e8c62ac
1 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
solenv/gbuild/platform/com_MSC_defs.mk
View File

@@ -275,6 +275,15 @@ 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_COMPILERNOOPTFLAGS := -Od