[ros-diffs] [tkreuzer] 56859: [CMAKE] Globally disable SSE when compiling with VS 11 or newer. See issue #7174 for more details.

Author: tkreuzer
Date: Sun Jul  8 19:59:33 2012
New Revision: 56859

URL: http://svn.reactos.org/svn/reactos?rev=56859&view=rev
Log:
[CMAKE]
Globally disable SSE when compiling with VS 11 or newer.

See issue #7174 for more details.

Modified:
    trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt
    trunk/reactos/cmake/msvc.cmake

Modified: trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/CMake…
==============================================================================
--- trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt [iso-8859-1] (original)
+++ trunk/reactos/boot/freeldr/freeldr/CMakeLists.txt [iso-8859-1] Sun Jul  8 19:59:33
2012
@@ -31,11 +31,6 @@
         add_definitions(-D_ZOOM2_)
     endif()
 endif()
-
-# for VS 2012 we need to explicitly disable SSE
-if (MSVC_VERSION GREATER 1699 AND ARCH MATCHES i386)
-        add_definitions(/arch:IA32)
-endif ()
 
 list(APPEND FREELDR_COMMON_SOURCE
     cmdline.c

Modified: trunk/reactos/cmake/msvc.cmake
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/msvc.cmake?rev=56859…
==============================================================================
--- trunk/reactos/cmake/msvc.cmake [iso-8859-1] (original)
+++ trunk/reactos/cmake/msvc.cmake [iso-8859-1] Sun Jul  8 19:59:33 2012
@@ -20,6 +20,12 @@
 add_definitions(/Dinline=__inline /D__STDC__=1)
 
 add_compile_flags("/X /GR- /GS- /Zl /W3")
+
+# HACK: for VS 11+ we need to explicitly disable SSE, which is off by
+# default for older compilers. See bug #7174
+if (MSVC_VERSION GREATER 1699 AND ARCH MATCHES i386)
+    add_compile_flags("/arch:IA32")
+endif ()
 
 # C4700 is almost always likely to result in broken code, so mark it as an error
 add_compile_flags("/we4700")