[ros-diffs] [reactos] 01/17: Revert "[CMAKE] Disable SSE instruction on GCC amd64 builds"

[Jérôme Gardou]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=b52ab2493b8af3bd09383…

commit b52ab2493b8af3bd0938380a67f0a903f0c3f60a
Author:     Jérôme Gardou <jerome.gardou(a)reactos.org>
AuthorDate: Wed Apr 28 17:17:02 2021 +0200
Commit:     Jérôme Gardou <zefklop(a)users.noreply.github.com>
CommitDate: Mon May 3 22:00:57 2021 +0200

    Revert "[CMAKE] Disable SSE instruction on GCC amd64 builds"
    
    The right fix is to ensure that the stack is properly 16-bit aligned
    This reverts commit 63944988a25335a5ac97f4028e759aede04a8458.
---
 sdk/cmake/gcc.cmake | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/sdk/cmake/gcc.cmake b/sdk/cmake/gcc.cmake
index 25893031e52..f542d34ef19 100644
--- a/sdk/cmake/gcc.cmake
+++ b/sdk/cmake/gcc.cmake
@@ -172,14 +172,6 @@ add_definitions(-D_inline=__inline)
 # Fix build with GLIBCXX + our c++ headers
 add_definitions(-D_GLIBCXX_HAVE_BROKEN_VSWPRINTF)
 
-# Disable SSE globally on amd64 builds.
-# We can't disable it for kernel mode modules only,
-# as we don't call set_module_type on static libraries
-# Instead we disable this options for user-mode modules
-if(ARCH STREQUAL "amd64")
-    add_compile_options(-mno-sse)
-endif()
-
 # Alternative arch name
 if(ARCH STREQUAL "amd64")
     set(ARCH2 x86_64)
@@ -322,9 +314,6 @@ function(set_module_type_toolchain MODULE TYPE)
 
         # Believe it or not, cmake doesn't do that
         set_property(TARGET ${MODULE} APPEND PROPERTY LINK_DEPENDS
$<TARGET_PROPERTY:native-pefixup,IMPORTED_LOCATION>)
-    else()
-        # We can safely use SSE for user-mode parts
-        remove_target_compile_option(${MODULE} -mno-sse)
     endif()
 endfunction()