Attachment 343821 Details for Bug 187127 – Patch

[patch] Patch

bug-187127-20180628114011.patch (text/plain), 4.75 KB, created by Ross Kirsling on 2018-06-28 11:40:12 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Ross Kirsling

Created: 2018-06-28 11:40:12 PDT

Size: 4.75 KB

patch

obsolete

>Subversion Revision: 233292
>diff --git a/Source/cmake/OptionsJSCOnly.cmake b/Source/cmake/OptionsJSCOnly.cmake
>index 3719d510e909fc432891ff99e20b66d50da32cf3..9252dbdf97e9f4399f04cd1e712a79846a655553 100644
>--- a/Source/cmake/OptionsJSCOnly.cmake
>+++ b/Source/cmake/OptionsJSCOnly.cmake
>@@ -17,8 +17,6 @@ WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_REMOTE_INSPECTOR PRIVATE OFF)
> WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_INTL_NUMBER_FORMAT_TO_PARTS PRIVATE ON)
> WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_INTL_PLURAL_RULES PRIVATE ON)
> if (WIN32)
>-    # FIXME: Enable FTL on Windows. https://bugs.webkit.org/show_bug.cgi?id=145366
>-    WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_FTL_JIT PRIVATE OFF)
>     # FIXME: Port bmalloc to Windows. https://bugs.webkit.org/show_bug.cgi?id=143310
>     WEBKIT_OPTION_DEFAULT_PORT_VALUE(USE_SYSTEM_MALLOC PRIVATE ON)
> endif ()
>@@ -73,8 +71,9 @@ if (WIN32)
>         set_property(GLOBAL PROPERTY FIND_LIBRARY_USE_LIB32_PATHS OFF)
>         set_property(GLOBAL PROPERTY FIND_LIBRARY_USE_LIB64_PATHS ON)
> 
>+        set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/lib64)
>         set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/lib64)
>-        set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
>+        set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin64)
>     endif ()
> 
>     set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY_DEBUG "${CMAKE_ARCHIVE_OUTPUT_DIRECTORY}")
>@@ -98,6 +97,9 @@ endif ()
> 
> if (NOT APPLE)
>     find_package(ICU REQUIRED)
>+    if (WIN32)
>+        add_definitions(-DUCHAR_TYPE=wchar_t)
>+    endif ()
> else ()
>     add_definitions(-DU_DISABLE_RENAMING=1 -DU_SHOW_CPLUSPLUS_API=0)
>     set(ICU_LIBRARIES libicucore.dylib)
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 5c6c081467055bba74e863407164b4ffa7887aeb..9ae06e2f5ad83fa40f69f0c894b6285b86857ede 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,19 @@
>+2018-06-28  Ross Kirsling  <ross.kirsling@sony.com>
>+
>+        [JSCOnly] Restore Windows build.
>+        https://bugs.webkit.org/show_bug.cgi?id=187127
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Scripts/build-jsc:
>+        Fix condition for disabling FTL JIT on Windows.
>+
>+        * Scripts/webkitdirs.pm:
>+        (executableProductDir):
>+        JSCOnly should still use bin64 on Windows (for consistency with full WebKit build).
>+        (determineIsWin64):
>+        JSCOnly already defaults to 64-bit on Windows with Ninja -- ensure that this is true even with MSBuild.
>+
> 2018-06-27  Daniel Bates  <dabates@apple.com>
> 
>         style-queue "AttributeError: 'NoneType' object has no attribute 'is_obsolete'" when processing security patch
>diff --git a/Tools/Scripts/build-jsc b/Tools/Scripts/build-jsc
>index 27e8b1326c8d5884f2f17dcc6e159b66b6fd88aa..f1def2ad915b0a61300da0d4822c88917bad7d7d 100755
>--- a/Tools/Scripts/build-jsc
>+++ b/Tools/Scripts/build-jsc
>@@ -115,7 +115,7 @@ if ($showHelp) {
>    exit 1;
> }
> 
>-if (isAppleWinWebKit()) {
>+if (isAnyWindows()) {
>     $ftlJIT = 0;
> }
> 
>diff --git a/Tools/Scripts/webkitdirs.pm b/Tools/Scripts/webkitdirs.pm
>index 909218a204435a7cccdebd671916f3f91ba70b1d..ebbe65fd854638b89e184ef25c24e214e56aa1fb 100755
>--- a/Tools/Scripts/webkitdirs.pm
>+++ b/Tools/Scripts/webkitdirs.pm
>@@ -815,10 +815,10 @@ sub executableProductDir
>     my $productDirectory = productDir();
> 
>     my $binaryDirectory;
>-    if (isGtk() || isJSCOnly() || isWPE()) {
>-        $binaryDirectory = "bin";
>-    } elsif (isAnyWindows()) {
>+    if (isAnyWindows()) {
>         $binaryDirectory = isWin64() ? "bin64" : "bin32";
>+    } elsif (isGtk() || isJSCOnly() || isWPE()) {
>+        $binaryDirectory = "bin";
>     } else {
>         return $productDirectory;
>     }
>@@ -1296,7 +1296,7 @@ sub isWin64()
> sub determineIsWin64()
> {
>     return if defined($isWin64);
>-    $isWin64 = checkForArgumentAndRemoveFromARGV("--64-bit") || isWinCairo() && !shouldBuild32Bit();
>+    $isWin64 = checkForArgumentAndRemoveFromARGV("--64-bit") || ((isWinCairo() || isJSCOnly) && !shouldBuild32Bit());
> }
> 
> sub determineIsWin64FromArchitecture($)
>diff --git a/ChangeLog b/ChangeLog
>index de4c55c3ffafd749635299b9f4127fe6e30fb0a6..f79b0ed58390424a2045726df0f5977fc9d74ba3 100644
>--- a/ChangeLog
>+++ b/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-06-28  Ross Kirsling  <ross.kirsling@sony.com>
>+
>+        [JSCOnly] Restore Windows build.
>+        https://bugs.webkit.org/show_bug.cgi?id=187127
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Source/cmake/OptionsJSCOnly.cmake:
>+        - Don't forget to set -DUCHAR_TYPE=wchar_t for ICU on Windows.
>+        - Use bin64/lib64 on Windows (for consistency with full WebKit build).
>+        - Remove misleading FTL JIT option (this does nothing because the setting in build-jsc has higher precedence).
>+
> 2018-06-18  Zan Dobersek  <zdobersek@igalia.com>
> 
>         Unreviewed GTK+ gardening. Updating the WPT expectations by removing

Actions: View | Formatted Diff | Diff

Attachments on bug 187127: 343821 | 343822 | 343838 | 343890 | 343892