[CMake] Clean up JSC JIT options
authormcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 23 Apr 2015 02:15:02 +0000 (02:15 +0000)
committermcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 23 Apr 2015 02:15:02 +0000 (02:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=143998

Reviewed by Filip Pizlo.

.:

* Source/cmake/OptionsEfl.cmake: Remove handling of ENABLE_LLINT_C_LOOP and preprocessor
  define for ENABLE_FTL_NATIVE_CALL_INLINING.
* Source/cmake/OptionsMac.cmake: Remove override of ENABLE_LLINT_C_LOOP
* Source/cmake/WebKitFeatures.cmake: Remove ENABLE_LLINT_C_LOOP. Add ENABLE_DFG_JIT and
  ENABLE_FTL_NATIVE_CALL_INLINING. Specify proper dependencies for these features.
* Source/cmakeconfig.h.cmake: Remove ENABLE_LLINT_C_LOOP and add ENABLE_DFG_JIT.

Tools:

* Scripts/build-jsc: Do not use -ENABLE_LLINT_C_LOOP on CMake ports.
* Scripts/webkitperl/FeatureList.pm: Remove ENABLE_LLINT_C_LOOP.

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

ChangeLog
Source/cmake/OptionsEfl.cmake
Source/cmake/OptionsMac.cmake
Source/cmake/WebKitFeatures.cmake
Source/cmakeconfig.h.cmake
Tools/ChangeLog
Tools/Scripts/build-jsc
Tools/Scripts/webkitperl/FeatureList.pm

index 341cea4..be72df8 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,19 @@
 2015-04-22  Michael Catanzaro  <mcatanzaro@igalia.com>
 
+        [CMake] Clean up JSC JIT options
+        https://bugs.webkit.org/show_bug.cgi?id=143998
+
+        Reviewed by Filip Pizlo.
+
+        * Source/cmake/OptionsEfl.cmake: Remove handling of ENABLE_LLINT_C_LOOP and preprocessor
+          define for ENABLE_FTL_NATIVE_CALL_INLINING.
+        * Source/cmake/OptionsMac.cmake: Remove override of ENABLE_LLINT_C_LOOP
+        * Source/cmake/WebKitFeatures.cmake: Remove ENABLE_LLINT_C_LOOP. Add ENABLE_DFG_JIT and
+          ENABLE_FTL_NATIVE_CALL_INLINING. Specify proper dependencies for these features.
+        * Source/cmakeconfig.h.cmake: Remove ENABLE_LLINT_C_LOOP and add ENABLE_DFG_JIT.
+
+2015-04-22  Michael Catanzaro  <mcatanzaro@igalia.com>
+
         [CMake] Use lowercase for local variables in macros
         https://bugs.webkit.org/show_bug.cgi?id=144059
 
index 9216a98..582c5fa 100644 (file)
@@ -124,13 +124,6 @@ WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_WEB_TIMING PUBLIC ON)
 WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_WEBGL PUBLIC ON)
 WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_XHR_TIMEOUT PUBLIC ON)
 
-if (ENABLE_LLINT_C_LOOP)
-    message(STATUS "Force enabling LLINT C LOOP.")
-    WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_JIT PUBLIC OFF)
-    WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_DFG_JIT PUBLIC OFF)
-    WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_FTL_JIT PUBLIC OFF)
-endif ()
-
 WEBKIT_OPTION_END()
 
 # Begin to find necessary packages for EFL port.
@@ -298,7 +291,6 @@ if (ENABLE_FTL_JIT)
     set(HAVE_LLVM ON)
     if (ENABLE_FTL_NATIVE_CALL_INLINING)
         find_package(Clang REQUIRED)
-        add_definitions(-DENABLE_FTL_NATIVE_CALL_INLINING=1)
     endif ()
 endif ()
 
index bc28b99..fd745ce 100644 (file)
@@ -213,8 +213,6 @@ WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_IOS_AIRPLAY PRIVATE OFF)
 WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_IOS_AIRPLAY_iphoneos PRIVATE OFF)
 WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_IOS_AIRPLAY_iphonesimulator PRIVATE OFF)
 
-WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_LLINT_C_LOOP PRIVATE OFF)
-
 WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_SANDBOX_EXTENSIONS PRIVATE ON)
 WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_WEB_PROCESS_SANDBOX PRIVATE ON)
 WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_NETWORK_PROCESS PRIVATE ON)
index 423852e..21ade43 100644 (file)
@@ -58,6 +58,7 @@ macro(WEBKIT_OPTION_BEGIN)
     WEBKIT_OPTION_DEFINE(ENABLE_DATA_TRANSFER_ITEMS "Toggle HTML5 data transfer items support" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_DETAILS_ELEMENT "Toggle HTML5 details support" PRIVATE ON)
     WEBKIT_OPTION_DEFINE(ENABLE_DEVICE_ORIENTATION "Toggle DeviceOrientation support" PRIVATE OFF)
+    WEBKIT_OPTION_DEFINE(ENABLE_DFG_JIT "Toggle data flow graph JIT tier" PRIVATE ON)
     WEBKIT_OPTION_DEFINE(ENABLE_DOM4_EVENTS_CONSTRUCTOR "Toggle DOM4 Events constructors" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_DOWNLOAD_ATTRIBUTE "Toggle download attribute support" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_DRAG_SUPPORT "Toggle Drag Support" PRIVATE OFF)
@@ -68,6 +69,7 @@ macro(WEBKIT_OPTION_BEGIN)
     WEBKIT_OPTION_DEFINE(ENABLE_FONT_LOAD_EVENTS "Toggle Font Load Events support" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_FTPDIR "Toggle FTP directory support" PRIVATE ON)
     WEBKIT_OPTION_DEFINE(ENABLE_FTL_JIT "Toggle FTL support for JSC" PRIVATE OFF)
+    WEBKIT_OPTION_DEFINE(ENABLE_FTL_NATIVE_CALL_INLINING "Toggle native call inlining for FTL" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_FULLSCREEN_API "Toggle Fullscreen API support" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_GAMEPAD "Toggle Gamepad support" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_GAMEPAD_DEPRECATED "Toggle deprecated Gamepad support" PRIVATE OFF)
@@ -94,7 +96,6 @@ macro(WEBKIT_OPTION_BEGIN)
     WEBKIT_OPTION_DEFINE(ENABLE_LEGACY_WEB_AUDIO "Toggle Legacy Web Audio support" PRIVATE ON)
     WEBKIT_OPTION_DEFINE(ENABLE_LETTERPRESS "Toggle letterpress support" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_LINK_PREFETCH "Toggle pre fetching support" PRIVATE OFF)
-    WEBKIT_OPTION_DEFINE(ENABLE_LLINT_C_LOOP "Force use of the llint c loop" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_MAC_LONG_PRESS "Toggle mac long press support" PRIVATE OFF)
     WEBKIT_OPTION_DEFINE(ENABLE_MATHML "Toggle MathML support" PRIVATE ON)
     WEBKIT_OPTION_DEFINE(ENABLE_MEDIA_CAPTURE "Toggle Media Capture support" PRIVATE OFF)
@@ -159,6 +160,9 @@ macro(WEBKIT_OPTION_BEGIN)
     WEBKIT_OPTION_DEFINE(ENABLE_THREADED_COMPOSITOR "Toggle threaded compositor support" PRIVATE OFF)
 
     WEBKIT_OPTION_DEPEND(ENABLE_ENCRYPTED_MEDIA_V2 ENABLE_VIDEO)
+    WEBKIT_OPTION_DEPEND(ENABLE_DFG_JIT ENABLE_JIT)
+    WEBKIT_OPTION_DEPEND(ENABLE_FTL_NATIVE_CALL_INLINING ENABLE_FTL_JIT)
+    WEBKIT_OPTION_DEPEND(ENABLE_FTL_JIT ENABLE_DFG_JIT)
     WEBKIT_OPTION_DEPEND(ENABLE_MEDIA_CONTROLS_SCRIPT ENABLE_VIDEO)
     WEBKIT_OPTION_DEPEND(ENABLE_VIDEO_TRACK ENABLE_VIDEO)
     WEBKIT_OPTION_DEPEND(ENABLE_TOUCH_SLIDER ENABLE_TOUCH_EVENTS)
index 877ef5c..966854b 100644 (file)
@@ -40,6 +40,7 @@
 #cmakedefine01 ENABLE_DATA_TRANSFER_ITEMS
 #cmakedefine01 ENABLE_DETAILS_ELEMENT
 #cmakedefine01 ENABLE_DEVICE_ORIENTATION
+#cmakedefine01 ENABLE_DFG_JIT
 #cmakedefine01 ENABLE_DOM4_EVENTS_CONSTRUCTOR
 #cmakedefine01 ENABLE_DOWNLOAD_ATTRIBUTE
 #cmakedefine01 ENABLE_DRAG_SUPPORT
@@ -76,7 +77,6 @@
 #cmakedefine01 ENABLE_LEGACY_VENDOR_PREFIXES
 #cmakedefine01 ENABLE_LEGACY_WEB_AUDIO
 #cmakedefine01 ENABLE_LINK_PREFETCH
-#cmakedefine01 ENABLE_LLINT_C_LOOP
 #cmakedefine01 ENABLE_MAC_LONG_PRESS
 #cmakedefine01 ENABLE_MATHML
 #cmakedefine01 ENABLE_MEDIA_CAPTURE
index 4c3a0ee..9dd6585 100644 (file)
@@ -1,3 +1,13 @@
+2015-04-22  Michael Catanzaro  <mcatanzaro@igalia.com>
+
+        [CMake] Clean up JSC JIT options
+        https://bugs.webkit.org/show_bug.cgi?id=143998
+
+        Reviewed by Filip Pizlo.
+
+        * Scripts/build-jsc: Do not use -ENABLE_LLINT_C_LOOP on CMake ports.
+        * Scripts/webkitperl/FeatureList.pm: Remove ENABLE_LLINT_C_LOOP.
+
 2015-04-21  Brent Fulgham  <bfulgham@apple.com>
 
         [Mac] Extend action menus to support PDF
index d55acf9..afe95c4 100755 (executable)
@@ -91,7 +91,7 @@ push @additionalSupportOptions, XcodeCoverageSupportOptions() if $coverageSuppor
 push @additionalSupportOptions, XcodeStaticAnalyzerOption() if $shouldRunStaticAnalyzer;
 
 if (cmakeBasedPortName()) {
-    $cmakeArgs .= " -DENABLE_LLINT_C_LOOP=ON" if $forceCLoop;
+    $cmakeArgs .= " -DENABLE_JIT=OFF" if $forceCLoop;
     $cmakeArgs .= " -DENABLE_FTL_JIT=ON" if $ftlJIT;
     # By default we build using all of the available CPUs
     $makeArgs .= ($makeArgs ? " " : "") . "-j" . numberOfCPUs() if $makeArgs !~ /-j\s*\d+/;
index 00032c1..93f82e6 100644 (file)
@@ -145,7 +145,6 @@ my (
     $xhrTimeoutSupport,
     $xsltSupport,
     $ftlJITSupport,
-    $forceCLoop,
 );
 
 my @features = (
@@ -433,9 +432,6 @@ my @features = (
 
     { option => "ftl-jit", desc => "Toggle FTLJIT support",
       define => "ENABLE_FTL_JIT", default => 0, value => \$ftlJITSupport },
-
-    { option => "cloop", desc => "Force use of the llint c loop",
-      define => "ENABLE_LLINT_C_LOOP", default => 0, value => \$forceCLoop },
 );
 
 sub getFeatureOptionList()