[EFL][CMake] Fix javascriptcore test failings by the jsCStack branch merge
authorgyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Feb 2014 05:32:01 +0000 (05:32 +0000)
committergyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Feb 2014 05:32:01 +0000 (05:32 +0000)
https://bugs.webkit.org/show_bug.cgi?id=128961

Reviewed by Ryosuke Niwa.

Javascriptcore test on EFL port has been broken since the jsCStack branch merge.
For now we can fix almost tests by using "-fno-tree-dce option". Unfortunately,
EFL port needs not to check "uninitialized" build warning as error in order to
adjust the option to EFL port. This patch don't take it as error temporarily.

* Source/cmake/OptionsEfl.cmake:
* Source/cmake/WebKitHelpers.cmake:

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

ChangeLog
Source/cmake/OptionsEfl.cmake
Source/cmake/WebKitHelpers.cmake

index 9a90fcf..eb6ed94 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2014-02-18  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
+
+        [EFL][CMake] Fix javascriptcore test failings by the jsCStack branch merge
+        https://bugs.webkit.org/show_bug.cgi?id=128961
+
+        Reviewed by Ryosuke Niwa.
+
+        Javascriptcore test on EFL port has been broken since the jsCStack branch merge.
+        For now we can fix almost tests by using "-fno-tree-dce option". Unfortunately,
+        EFL port needs not to check "uninitialized" build warning as error in order to
+        adjust the option to EFL port. This patch don't take it as error temporarily.
+
+        * Source/cmake/OptionsEfl.cmake:
+        * Source/cmake/WebKitHelpers.cmake:
+
 2014-02-16  Martin Robinson  <mrobinson@igalia.com>
 
         [GTK] [CMake] JavaScriptCore crashes in JSC::eval
 2014-02-16  Martin Robinson  <mrobinson@igalia.com>
 
         [GTK] [CMake] JavaScriptCore crashes in JSC::eval
index 85e5665..c1d892e 100644 (file)
@@ -264,8 +264,8 @@ endif ()
 
 # push of rbp is needed after JSC JIT uses CStack
 if (CMAKE_COMPILER_IS_GNUCC AND UNIX AND NOT APPLE)
 
 # push of rbp is needed after JSC JIT uses CStack
 if (CMAKE_COMPILER_IS_GNUCC AND UNIX AND NOT APPLE)
-    set(CMAKE_C_FLAGS_RELEASE "-fno-omit-frame-pointer ${CMAKE_C_FLAGS_RELEASE}")
-    set(CMAKE_CXX_FLAGS_RELEASE "-fno-omit-frame-pointer ${CMAKE_CXX_FLAGS_RELEASE}")
+    set(CMAKE_C_FLAGS_RELEASE "-fno-omit-frame-pointer -fno-tree-dce ${CMAKE_C_FLAGS_RELEASE}")
+    set(CMAKE_CXX_FLAGS_RELEASE "-fno-omit-frame-pointer -fno-tree-dce ${CMAKE_CXX_FLAGS_RELEASE}")
 endif ()
 
 
 endif ()
 
 
index 409931c..5d08adc 100644 (file)
@@ -44,7 +44,9 @@ macro(WEBKIT_SET_EXTRA_COMPILER_FLAGS _target)
 
         # Enable errors on warning
         if (OPTION_ENABLE_WERROR)
 
         # Enable errors on warning
         if (OPTION_ENABLE_WERROR)
-            set(OLD_COMPILE_FLAGS "-Werror -Wno-error=unused-parameter ${OLD_COMPILE_FLAGS}")
+            # FIXME: When we use -fno-tree-dce to support the jsCStack branch merge, build error occurs due to the uninitialization. Temporarily we set
+            # uninitialized as build warning in order to support the jsCStack merge. https://bugs.webkit.org/show_bug.cgi?id=127777.
+            set(OLD_COMPILE_FLAGS "-Werror -Wno-error=unused-parameter -Wno-error=uninitialized ${OLD_COMPILE_FLAGS}")
         endif ()
 
         set_target_properties(${_target} PROPERTIES
         endif ()
 
         set_target_properties(${_target} PROPERTIES