[CMake] Do not use a --no-keep-memory linker flag in 64bit system
authoryoon@igalia.com <yoon@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 22 May 2017 13:46:41 +0000 (13:46 +0000)
committeryoon@igalia.com <yoon@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 22 May 2017 13:46:41 +0000 (13:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=172429

Reviewed by Michael Catanzaro.

* Source/cmake/OptionsCommon.cmake:
In x86_64, cmake accidentally applies --no-keep-memory since MATCHES
uses the given value as a regex. It reduces linking performance
due to the frequent disk operations in x86_64.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@217218 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Source/cmake/OptionsCommon.cmake

index 6cdc6c6..1d021fa 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2017-05-22  Gwang Yoon Hwang  <yoon@igalia.com>
+
+        [CMake] Do not use a --no-keep-memory linker flag in 64bit system
+        https://bugs.webkit.org/show_bug.cgi?id=172429
+
+        Reviewed by Michael Catanzaro.
+
+        * Source/cmake/OptionsCommon.cmake:
+        In x86_64, cmake accidentally applies --no-keep-memory since MATCHES
+        uses the given value as a regex. It reduces linking performance
+        due to the frequent disk operations in x86_64.
+
 2017-05-22  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         Unreviewed. Update OptionsGTK.cmake and NEWS for 2.17.3 release.
index d32ed7d..e77c707 100644 (file)
@@ -173,7 +173,7 @@ if (COMPILER_IS_CLANG)
 endif ()
 
 string(TOLOWER ${CMAKE_HOST_SYSTEM_PROCESSOR} LOWERCASE_CMAKE_HOST_SYSTEM_PROCESSOR)
-if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU" AND "${LOWERCASE_CMAKE_HOST_SYSTEM_PROCESSOR}" MATCHES "(i[3-6]86|x86|armhf)")
+if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU" AND NOT "${LOWERCASE_CMAKE_HOST_SYSTEM_PROCESSOR}" MATCHES "x86_64")
     # To avoid out of memory when building with debug option in 32bit system.
     # See https://bugs.webkit.org/show_bug.cgi?id=77327
     set(CMAKE_SHARED_LINKER_FLAGS_DEBUG "-Wl,--no-keep-memory ${CMAKE_SHARED_LINKER_FLAGS_DEBUG}")