libv8 6.3.292.48.1 → 6.7.288.46.0

data/ext/libv8/compiler.rb

@@ -1,39 +0,0 @@
-require 'rbconfig'
-require File.expand_path '../compiler/generic_compiler', __FILE__
-require File.expand_path '../compiler/gcc', __FILE__
-require File.expand_path '../compiler/clang', __FILE__
-require File.expand_path '../compiler/apple_llvm', __FILE__
-
-module Libv8
-  module Compiler
-    module_function
-
-    def type_of(compiler)
-      case version_string_of(compiler)
-      when /^Apple LLVM\b/ then AppleLLVM
-      when /\bclang\b/i then Clang
-      when /^gcc/i then GCC
-      else GenericCompiler
-      end
-    end
-
-    def version_string_of(compiler)
-      command_result = execute_command "env LC_ALL=C LANG=C #{compiler} -v 2>&1"
-
-      unless command_result.status.success?
-        raise "Could not get version string of compiler #{compiler}"
-      end
-
-      command_result.output
-    end
-
-    def execute_command(command)
-      output = `#{command}`
-      status = $?
-      ExecutionResult.new output, status
-    end
-
-    class ExecutionResult < Struct.new(:output, :status)
-    end
-  end
-end

data/ext/libv8/compiler/apple_llvm.rb

@@ -1,22 +0,0 @@
-module Libv8
-  module Compiler
-    class AppleLLVM < Clang
-      LLVM_VERSION_REGEXP = /Apple LLVM version (\d+\.\d+(\.\d+)*) \(/i
-      REQUIRED_VERSION = '4.3'
-
-      def name
-        'Apple LLVM'
-      end
-
-      private
-
-      def required_version
-        REQUIRED_VERSION
-      end
-
-      def version_regexp
-        LLVM_VERSION_REGEXP
-      end
-    end
-  end
-end

data/ext/libv8/compiler/clang.rb

@@ -1,22 +0,0 @@
-module Libv8
-  module Compiler
-    class Clang < GenericCompiler
-      CLANG_VERSION_REGEXP = /clang version (\d+\.\d+(\.\d+)*) \(/i
-      REQUIRED_VERSION = '3.1'
-
-      def name
-        'clang'
-      end
-
-      private
-
-      def required_version
-        REQUIRED_VERSION
-      end
-
-      def version_regexp
-        CLANG_VERSION_REGEXP
-      end
-    end
-  end
-end

data/ext/libv8/compiler/gcc.rb

@@ -1,22 +0,0 @@
-module Libv8
-  module Compiler
-    class GCC < GenericCompiler
-      GCC_VERSION_REGEXP = /gcc version (\d+\.\d+(\.\d+)*)/i
-      REQUIRED_VERSION = '4.7'
-
-      def name
-        'GCC'
-      end
-
-      private
-
-      def required_version
-        REQUIRED_VERSION
-      end
-
-      def version_regexp
-        GCC_VERSION_REGEXP
-      end
-    end
-  end
-end

data/ext/libv8/compiler/generic_compiler.rb

@@ -1,66 +0,0 @@
-module Libv8
-  module Compiler
-    class GenericCompiler
-      GENERIC_VERSION_REGEXP = /(\d+\.\d+(\.\d+)*)/
-      GENERIC_TARGET_REGEXP = /Target: ([a-z0-9\-_.]*)/
-
-      def initialize(command)
-        @command = command
-      end
-
-      def name
-        File.basename @command
-      end
-
-      def to_s
-        @command
-      end
-
-      def version
-        version_string =~ version_regexp
-        $1
-      end
-
-      def target
-        version_string =~ target_regexp
-        $1
-      end
-
-      def compatible?
-        return false unless required_version && !version.nil?
-
-        (string_to_semver(version) <=> string_to_semver(required_version)) > -1
-      end
-
-      def call(*arguments)
-        Compiler::execute_command arguments.unshift(@command).join(' ')
-      end
-
-      private
-
-      def version_string
-        begin
-          Compiler::version_string_of @command
-        rescue StandardError
-          nil
-        end
-      end
-
-      def version_regexp
-        GENERIC_VERSION_REGEXP
-      end
-
-      def target_regexp
-        GENERIC_TARGET_REGEXP
-      end
-
-      def required_version
-        nil
-      end
-
-      def string_to_semver(version)
-        version.split('.').map(&:to_i)
-      end
-    end
-  end
-end

data/ext/libv8/make.rb

@@ -1,13 +0,0 @@
-module Libv8
-  module Make
-    module_function
-
-    def make
-      unless defined?(@make)
-        @make = `which gmake 2> /dev/null`.chomp
-        @make = `which make`.chomp unless $?.success?
-      end
-      @make
-    end
-  end
-end

data/ext/libv8/patcher.rb

@@ -1,21 +0,0 @@
-module Libv8
-  module Patcher
-    PATCH_DIRECTORY = File.expand_path '../../../patches', __FILE__
-
-    module_function
-
-    def patch!
-      File.open(".applied_patches", File::RDWR|File::CREAT) do |f|
-        available_patches = Dir.glob(File.join(PATCH_DIRECTORY, '*.patch')).sort
-        applied_patches = f.readlines.map(&:chomp)
-
-        (available_patches - applied_patches).each do |patch|
-          puts "Applying #{patch}"
-          `patch -p1 -N < #{patch}`
-          fail "failed to apply patch #{patch}" unless $?.success?
-          f.puts patch
-        end
-      end
-    end
-  end
-end

data/patches/0001-Build-a-standalone-static-library.patch

@@ -1,26 +0,0 @@
-From a018c0c78b9d40c9d9787fe969530e40fca5d38a Mon Sep 17 00:00:00 2001
-From: Petko Bordjukov <bordjukov@gmail.com>
-Date: Mon, 25 Jul 2016 00:00:19 +0300
-Subject: [PATCH 1/5] Build a standalone static library
-
----
- gypfiles/standalone.gypi | 3 +++
- 1 file changed, 3 insertions(+)
-
-diff --git a/gypfiles/standalone.gypi b/gypfiles/standalone.gypi
-index 63930d8aef..d0d00d3f67 100644
---- a/gypfiles/standalone.gypi
-+++ b/gypfiles/standalone.gypi
-@@ -517,6 +517,9 @@
-       }],  # fastbuild!=0
-     ],
-     'target_conditions': [
-+      ['_type=="static_library"', {
-+        'standalone_static_library': 1,
-+      }],
-       ['v8_code == 0', {
-         'defines!': [
-           'DEBUG',
--- 
-2.15.1
-

data/patches/0002-Don-t-compile-unnecessary-stuff.patch

@@ -1,85 +0,0 @@
-From 9ec36a77bb241ee4532553809b1ec7281ea998b1 Mon Sep 17 00:00:00 2001
-From: Petko Bordjukov <bordjukov@gmail.com>
-Date: Fri, 28 Jul 2017 11:11:08 +0300
-Subject: [PATCH 2/5] Don't compile unnecessary stuff
-
----
- Makefile         |  9 ++-------
- gypfiles/all.gyp | 33 ---------------------------------
- 2 files changed, 2 insertions(+), 40 deletions(-)
-
-diff --git a/Makefile b/Makefile
-index 167ebf8c08..d9d0b9b306 100644
---- a/Makefile
-+++ b/Makefile
-@@ -268,14 +268,9 @@ ANDROID_ARCHES = android_ia32 android_x64 android_arm android_arm64 \
- 		 android_mipsel
- 
- # List of files that trigger Makefile regeneration:
--GYPFILES = third_party/icu/icu.gypi third_party/icu/icu.gyp \
--	   gypfiles/shim_headers.gypi gypfiles/features.gypi \
--           gypfiles/standalone.gypi \
-+GYPFILES = gypfiles/features.gypi gypfiles/standalone.gypi \
- 	   gypfiles/toolchain.gypi gypfiles/all.gyp gypfiles/mac/asan.gyp \
--	   test/cctest/cctest.gyp test/fuzzer/fuzzer.gyp \
--	   test/unittests/unittests.gyp src/v8.gyp \
--	   tools/parser-shell.gyp testing/gmock.gyp testing/gtest.gyp \
--	   samples/samples.gyp src/third_party/vtune/v8vtune.gyp src/d8.gyp
-+	   src/v8.gyp src/d8.gyp
- 
- # If vtunejit=on, the v8vtune.gyp will be appended.
- ifeq ($(vtunejit), on)
-diff --git a/gypfiles/all.gyp b/gypfiles/all.gyp
-index bc9d9650eb..96820a0ecc 100644
---- a/gypfiles/all.gyp
-+++ b/gypfiles/all.gyp
-@@ -9,8 +9,6 @@
-       'type': 'none',
-       'dependencies': [
-         '../src/d8.gyp:d8',
--        '../test/inspector/inspector.gyp:*',
--        '../test/mkgrokdump/mkgrokdump.gyp:*',
-       ],
-       'conditions': [
-         ['component!="shared_library"', {
-@@ -18,37 +16,6 @@
-             '../tools/parser-shell.gyp:parser-shell',
-           ],
-         }],
--        # These items don't compile for Android on Mac.
--        ['host_os!="mac" or OS!="android"', {
--          'dependencies': [
--            '../samples/samples.gyp:*',
--            '../test/cctest/cctest.gyp:*',
--            '../test/fuzzer/fuzzer.gyp:*',
--            '../test/unittests/unittests.gyp:*',
--          ],
--        }],
--        ['test_isolation_mode != "noop"', {
--          'dependencies': [
--            '../test/bot_default.gyp:*',
--            '../test/benchmarks/benchmarks.gyp:*',
--            '../test/debugger/debugger.gyp:*',
--            '../test/default.gyp:*',
--            '../test/intl/intl.gyp:*',
--            '../test/message/message.gyp:*',
--            '../test/mjsunit/mjsunit.gyp:*',
--            '../test/mozilla/mozilla.gyp:*',
--            '../test/optimize_for_size.gyp:*',
--            '../test/perf.gyp:*',
--            '../test/preparser/preparser.gyp:*',
--            '../test/test262/test262.gyp:*',
--            '../test/webkit/webkit.gyp:*',
--            '../tools/check-static-initializers.gyp:*',
--            '../tools/gcmole/run_gcmole.gyp:*',
--            '../tools/jsfunfuzz/jsfunfuzz.gyp:*',
--            '../tools/run-deopt-fuzzer.gyp:*',
--            '../tools/run-valgrind.gyp:*',
--          ],
--        }],
-       ]
-     }
-   ]
--- 
-2.15.1
-

data/patches/0003-Use-the-fPIC-flag-for-the-static-library.patch

@@ -1,25 +0,0 @@
-From 68eadedb9728ee6ce634d646835ef0aab2d23be6 Mon Sep 17 00:00:00 2001
-From: Petko Bordjukov <bordjukov@gmail.com>
-Date: Mon, 25 Jul 2016 00:05:47 +0300
-Subject: [PATCH 3/5] Use the -fPIC flag for the static library
-
----
- gypfiles/standalone.gypi | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/gypfiles/standalone.gypi b/gypfiles/standalone.gypi
-index d0d00d3f67..f01bd48596 100644
---- a/gypfiles/standalone.gypi
-+++ b/gypfiles/standalone.gypi
-@@ -797,7 +797,7 @@
-           [ 'visibility=="hidden" and v8_enable_backtrace==0', {
-             'cflags': [ '-fvisibility=hidden' ],
-           }],
--          [ 'component=="shared_library"', {
-+          [ 'component=="shared_library" or component=="static_library" and (v8_target_arch=="x64" or v8_target_arch=="arm64" or v8_target_arch=="arm")', {
-             'cflags': [ '-fPIC', ],
-           }],
-           [ 'clang==0 and coverage==1', {
--- 
-2.15.1
-

data/patches/0004-Do-not-embed-debug-symbols-in-macOS-libraries.patch

@@ -1,25 +0,0 @@
-From 943282d83340d51a7749f59708e5dca5bbe96099 Mon Sep 17 00:00:00 2001
-From: Jb Aviat <jb@sqreen.io>
-Date: Mon, 27 Feb 2017 11:14:42 +0200
-Subject: [PATCH 4/5] Do not embed debug symbols in macOS libraries
-
-This lowers dramatically the produced binary size
----
- gypfiles/standalone.gypi | 1 +
- 1 file changed, 1 insertion(+)
-
-diff --git a/gypfiles/standalone.gypi b/gypfiles/standalone.gypi
-index f01bd48596..25ca6b8606 100644
---- a/gypfiles/standalone.gypi
-+++ b/gypfiles/standalone.gypi
-@@ -1056,6 +1056,7 @@
-           'GCC_ENABLE_CPP_EXCEPTIONS': 'NO',        # -fno-exceptions
-           'GCC_ENABLE_CPP_RTTI': 'NO',              # -fno-rtti
-           'GCC_ENABLE_PASCAL_STRINGS': 'NO',        # No -mpascal-strings
-+          'GCC_GENERATE_DEBUGGING_SYMBOLS': 'NO',   # No -gdwarf-2
-           # GCC_INLINES_ARE_PRIVATE_EXTERN maps to -fvisibility-inlines-hidden
-           'GCC_INLINES_ARE_PRIVATE_EXTERN': 'YES',
-           'GCC_SYMBOLS_PRIVATE_EXTERN': 'YES',      # -fvisibility=hidden
--- 
-2.15.1
-

data/patches/0005-Remove-TryInstallOptimizedCode.patch

@@ -1,321 +0,0 @@
-From 723ed88782dbe9462aba5711a70be628d694bfda Mon Sep 17 00:00:00 2001
-From: Leszek Swirski <leszeks@chromium.org>
-Date: Tue, 17 Oct 2017 17:20:15 +0100
-Subject: [PATCH 5/5] Remove TryInstallOptimizedCode
-
-Removes the interrupt check and runtime call to TryInstallOptimizedCode
-from the optimization marker checks (i.e. CompileLazy and
-InterpreterEntryTrampoline). Instead, we rely on the other interrupt
-sources (in particular stack checks at function entries and loop
-headers) to install optimized code for us.
-
-This will hopefully not cause regressions, as we have plenty of other
-interrupt checks, but it may delay optimized code execution for
-some function by one function call.
-
-Bug: v8:6933
-Change-Id: Ieadfff7ae2078d2a84085294158ad9a706eb9c64
-Reviewed-on: https://chromium-review.googlesource.com/723475
-Reviewed-by: Ross McIlroy <rmcilroy@chromium.org>
-Commit-Queue: Leszek Swirski <leszeks@chromium.org>
-Cr-Commit-Position: refs/heads/master@{#48667}
----
- src/builtins/arm/builtins-arm.cc       | 13 +++----------
- src/builtins/arm64/builtins-arm64.cc   | 13 +++----------
- src/builtins/ia32/builtins-ia32.cc     | 15 +++------------
- src/builtins/mips/builtins-mips.cc     | 13 +++----------
- src/builtins/mips64/builtins-mips64.cc | 13 +++----------
- src/builtins/ppc/builtins-ppc.cc       | 14 +++-----------
- src/builtins/s390/builtins-s390.cc     | 13 +++----------
- src/builtins/x64/builtins-x64.cc       | 13 +++----------
- src/runtime/runtime-compiler.cc        | 21 ---------------------
- src/runtime/runtime.h                  |  1 -
- 10 files changed, 24 insertions(+), 105 deletions(-)
-
-diff --git a/src/builtins/arm/builtins-arm.cc b/src/builtins/arm/builtins-arm.cc
-index bf359d69e9..e8fa690660 100644
---- a/src/builtins/arm/builtins-arm.cc
-+++ b/src/builtins/arm/builtins-arm.cc
-@@ -782,22 +782,15 @@ static void MaybeTailCallOptimizedCodeSlot(MacroAssembler* masm,
-         Runtime::kCompileOptimized_Concurrent);
- 
-     {
--      // Otherwise, the marker is InOptimizationQueue.
-+      // Otherwise, the marker is InOptimizationQueue, so fall through hoping
-+      // that an interrupt will eventually update the slot with optimized code.
-       if (FLAG_debug_code) {
-         __ cmp(
-             optimized_code_entry,
-             Operand(Smi::FromEnum(OptimizationMarker::kInOptimizationQueue)));
-         __ Assert(eq, kExpectedOptimizationSentinel);
-       }
--      // Checking whether the queued function is ready for install is
--      // optional, since we come across interrupts and stack checks elsewhere.
--      // However, not checking may delay installing ready functions, and
--      // always checking would be quite expensive.  A good compromise is to
--      // first check against stack limit as a cue for an interrupt signal.
--      __ LoadRoot(scratch2, Heap::kStackLimitRootIndex);
--      __ cmp(sp, Operand(scratch2));
--      __ b(hs, &fallthrough);
--      GenerateTailCallToReturnedCode(masm, Runtime::kTryInstallOptimizedCode);
-+      __ jmp(&fallthrough);
-     }
-   }
- 
-diff --git a/src/builtins/arm64/builtins-arm64.cc b/src/builtins/arm64/builtins-arm64.cc
-index b1d5d32b9a..7aaa2d0003 100644
---- a/src/builtins/arm64/builtins-arm64.cc
-+++ b/src/builtins/arm64/builtins-arm64.cc
-@@ -788,22 +788,15 @@ static void MaybeTailCallOptimizedCodeSlot(MacroAssembler* masm,
-         Runtime::kCompileOptimized_Concurrent);
- 
-     {
--      // Otherwise, the marker is InOptimizationQueue.
-+      // Otherwise, the marker is InOptimizationQueue, so fall through hoping
-+      // that an interrupt will eventually update the slot with optimized code.
-       if (FLAG_debug_code) {
-         __ Cmp(
-             optimized_code_entry,
-             Operand(Smi::FromEnum(OptimizationMarker::kInOptimizationQueue)));
-         __ Assert(eq, kExpectedOptimizationSentinel);
-       }
--
--      // Checking whether the queued function is ready for install is optional,
--      // since we come across interrupts and stack checks elsewhere.  However,
--      // not checking may delay installing ready functions, and always checking
--      // would be quite expensive.  A good compromise is to first check against
--      // stack limit as a cue for an interrupt signal.
--      __ CompareRoot(masm->StackPointer(), Heap::kStackLimitRootIndex);
--      __ B(hs, &fallthrough);
--      GenerateTailCallToReturnedCode(masm, Runtime::kTryInstallOptimizedCode);
-+      __ B(&fallthrough);
-     }
-   }
- 
-diff --git a/src/builtins/ia32/builtins-ia32.cc b/src/builtins/ia32/builtins-ia32.cc
-index ee15025520..a689c3131d 100644
---- a/src/builtins/ia32/builtins-ia32.cc
-+++ b/src/builtins/ia32/builtins-ia32.cc
-@@ -698,24 +698,15 @@ static void MaybeTailCallOptimizedCodeSlot(MacroAssembler* masm,
-         Runtime::kCompileOptimized_Concurrent);
- 
-     {
--      // Otherwise, the marker is InOptimizationQueue.
-+      // Otherwise, the marker is InOptimizationQueue, so fall through hoping
-+      // that an interrupt will eventually update the slot with optimized code.
-       if (FLAG_debug_code) {
-         __ cmp(
-             optimized_code_entry,
-             Immediate(Smi::FromEnum(OptimizationMarker::kInOptimizationQueue)));
-         __ Assert(equal, kExpectedOptimizationSentinel);
-       }
--
--      // Checking whether the queued function is ready for install is optional,
--      // since we come across interrupts and stack checks elsewhere.  However,
--      // not checking may delay installing ready functions, and always checking
--      // would be quite expensive.  A good compromise is to first check against
--      // stack limit as a cue for an interrupt signal.
--      ExternalReference stack_limit =
--          ExternalReference::address_of_stack_limit(masm->isolate());
--      __ cmp(esp, Operand::StaticVariable(stack_limit));
--      __ j(above_equal, &fallthrough);
--      GenerateTailCallToReturnedCode(masm, Runtime::kTryInstallOptimizedCode);
-+      __ jmp(&fallthrough);
-     }
-   }
- 
-diff --git a/src/builtins/mips/builtins-mips.cc b/src/builtins/mips/builtins-mips.cc
-index e8f846c10a..4835fb0b1b 100644
---- a/src/builtins/mips/builtins-mips.cc
-+++ b/src/builtins/mips/builtins-mips.cc
-@@ -760,21 +760,14 @@ static void MaybeTailCallOptimizedCodeSlot(MacroAssembler* masm,
-         Runtime::kCompileOptimized_Concurrent);
- 
-     {
--      // Otherwise, the marker is InOptimizationQueue.
-+      // Otherwise, the marker is InOptimizationQueue, so fall through hoping
-+      // that an interrupt will eventually update the slot with optimized code.
-       if (FLAG_debug_code) {
-         __ Assert(
-             eq, kExpectedOptimizationSentinel, optimized_code_entry,
-             Operand(Smi::FromEnum(OptimizationMarker::kInOptimizationQueue)));
-       }
--
--      // Checking whether the queued function is ready for install is optional,
--      // since we come across interrupts and stack checks elsewhere. However,
--      // not checking may delay installing ready functions, and always checking
--      // would be quite expensive.  A good compromise is to first check against
--      // stack limit as a cue for an interrupt signal.
--      __ LoadRoot(at, Heap::kStackLimitRootIndex);
--      __ Branch(&fallthrough, hs, sp, Operand(at));
--      GenerateTailCallToReturnedCode(masm, Runtime::kTryInstallOptimizedCode);
-+      __ jmp(&fallthrough);
-     }
-   }
- 
-diff --git a/src/builtins/mips64/builtins-mips64.cc b/src/builtins/mips64/builtins-mips64.cc
-index f62750b061..2584444f1f 100644
---- a/src/builtins/mips64/builtins-mips64.cc
-+++ b/src/builtins/mips64/builtins-mips64.cc
-@@ -760,21 +760,14 @@ static void MaybeTailCallOptimizedCodeSlot(MacroAssembler* masm,
-         Runtime::kCompileOptimized_Concurrent);
- 
-     {
--      // Otherwise, the marker is InOptimizationQueue.
-+      // Otherwise, the marker is InOptimizationQueue, so fall through hoping
-+      // that an interrupt will eventually update the slot with optimized code.
-       if (FLAG_debug_code) {
-         __ Assert(
-             eq, kExpectedOptimizationSentinel, optimized_code_entry,
-             Operand(Smi::FromEnum(OptimizationMarker::kInOptimizationQueue)));
-       }
--
--      // Checking whether the queued function is ready for install is optional,
--      // since we come across interrupts and stack checks elsewhere. However,
--      // not checking may delay installing ready functions, and always checking
--      // would be quite expensive.  A good compromise is to first check against
--      // stack limit as a cue for an interrupt signal.
--      __ LoadRoot(t0, Heap::kStackLimitRootIndex);
--      __ Branch(&fallthrough, hs, sp, Operand(t0));
--      GenerateTailCallToReturnedCode(masm, Runtime::kTryInstallOptimizedCode);
-+      __ jmp(&fallthrough);
-     }
-   }
- 
-diff --git a/src/builtins/ppc/builtins-ppc.cc b/src/builtins/ppc/builtins-ppc.cc
-index 3ed3eb686d..c242be5cf8 100644
---- a/src/builtins/ppc/builtins-ppc.cc
-+++ b/src/builtins/ppc/builtins-ppc.cc
-@@ -780,23 +780,15 @@ static void MaybeTailCallOptimizedCodeSlot(MacroAssembler* masm,
-         Runtime::kCompileOptimized_Concurrent);
- 
-     {
--      // Otherwise, the marker is InOptimizationQueue.
-+      // Otherwise, the marker is InOptimizationQueue, so fall through hoping
-+      // that an interrupt will eventually update the slot with optimized code.
-       if (FLAG_debug_code) {
-         __ CmpSmiLiteral(
-             optimized_code_entry,
-             Smi::FromEnum(OptimizationMarker::kInOptimizationQueue), r0);
-         __ Assert(eq, kExpectedOptimizationSentinel);
-       }
--
--      // Checking whether the queued function is ready for install is optional,
--      // since we come across interrupts and stack checks elsewhere.  However,
--      // not checking may delay installing ready functions, and always checking
--      // would be quite expensive.  A good compromise is to first check against
--      // stack limit as a cue for an interrupt signal.
--      __ LoadRoot(ip, Heap::kStackLimitRootIndex);
--      __ cmpl(sp, ip);
--      __ bge(&fallthrough);
--      GenerateTailCallToReturnedCode(masm, Runtime::kTryInstallOptimizedCode);
-+      __ b(&fallthrough);
-     }
-   }
- 
-diff --git a/src/builtins/s390/builtins-s390.cc b/src/builtins/s390/builtins-s390.cc
-index e9ef390c69..aa9e62f217 100644
---- a/src/builtins/s390/builtins-s390.cc
-+++ b/src/builtins/s390/builtins-s390.cc
-@@ -783,22 +783,15 @@ static void MaybeTailCallOptimizedCodeSlot(MacroAssembler* masm,
-         Runtime::kCompileOptimized_Concurrent);
- 
-     {
--      // Otherwise, the marker is InOptimizationQueue.
-+      // Otherwise, the marker is InOptimizationQueue, so fall through hoping
-+      // that an interrupt will eventually update the slot with optimized code.
-       if (FLAG_debug_code) {
-         __ CmpSmiLiteral(
-             optimized_code_entry,
-             Smi::FromEnum(OptimizationMarker::kInOptimizationQueue), r0);
-         __ Assert(eq, kExpectedOptimizationSentinel);
-       }
--
--      // Checking whether the queued function is ready for install is optional,
--      // since we come across interrupts and stack checks elsewhere.  However,
--      // not checking may delay installing ready functions, and always checking
--      // would be quite expensive.  A good compromise is to first check against
--      // stack limit as a cue for an interrupt signal.
--      __ CmpLogicalP(sp, RootMemOperand(Heap::kStackLimitRootIndex));
--      __ bge(&fallthrough, Label::kNear);
--      GenerateTailCallToReturnedCode(masm, Runtime::kTryInstallOptimizedCode);
-+      __ b(&fallthrough, Label::kNear);
-     }
-   }
- 
-diff --git a/src/builtins/x64/builtins-x64.cc b/src/builtins/x64/builtins-x64.cc
-index 713475cd34..81c92681d5 100644
---- a/src/builtins/x64/builtins-x64.cc
-+++ b/src/builtins/x64/builtins-x64.cc
-@@ -781,21 +781,14 @@ static void MaybeTailCallOptimizedCodeSlot(MacroAssembler* masm,
-         Runtime::kCompileOptimized_Concurrent);
- 
-     {
--      // Otherwise, the marker is InOptimizationQueue.
-+      // Otherwise, the marker is InOptimizationQueue, so fall through hoping
-+      // that an interrupt will eventually update the slot with optimized code.
-       if (FLAG_debug_code) {
-         __ SmiCompare(optimized_code_entry,
-                       Smi::FromEnum(OptimizationMarker::kInOptimizationQueue));
-         __ Assert(equal, kExpectedOptimizationSentinel);
-       }
--
--      // Checking whether the queued function is ready for install is optional,
--      // since we come across interrupts and stack checks elsewhere.  However,
--      // not checking may delay installing ready functions, and always checking
--      // would be quite expensive.  A good compromise is to first check against
--      // stack limit as a cue for an interrupt signal.
--      __ CompareRoot(rsp, Heap::kStackLimitRootIndex);
--      __ j(above_equal, &fallthrough);
--      GenerateTailCallToReturnedCode(masm, Runtime::kTryInstallOptimizedCode);
-+      __ jmp(&fallthrough);
-     }
-   }
- 
-diff --git a/src/runtime/runtime-compiler.cc b/src/runtime/runtime-compiler.cc
-index 1cc00f5b7e..b445037d08 100644
---- a/src/runtime/runtime-compiler.cc
-+++ b/src/runtime/runtime-compiler.cc
-@@ -302,27 +302,6 @@ RUNTIME_FUNCTION(Runtime_CompileForOnStackReplacement) {
-   return NULL;
- }
- 
--
--RUNTIME_FUNCTION(Runtime_TryInstallOptimizedCode) {
--  HandleScope scope(isolate);
--  DCHECK_EQ(1, args.length());
--  CONVERT_ARG_HANDLE_CHECKED(JSFunction, function, 0);
--
--  // First check if this is a real stack overflow.
--  StackLimitCheck check(isolate);
--  if (check.JsHasOverflowed(kStackSpaceRequiredForCompilation * KB)) {
--    return isolate->StackOverflow();
--  }
--
--  // Only try to install optimized functions if the interrupt was InstallCode.
--  if (isolate->stack_guard()->CheckAndClearInstallCode()) {
--    isolate->optimizing_compile_dispatcher()->InstallOptimizedFunctions();
--  }
--
--  return (function->IsOptimized()) ? function->code()
--                                   : function->shared()->code();
--}
--
- static Object* CompileGlobalEval(Isolate* isolate, Handle<String> source,
-                                  Handle<SharedFunctionInfo> outer_info,
-                                  LanguageMode language_mode,
-diff --git a/src/runtime/runtime.h b/src/runtime/runtime.h
-index e7084a8cca..a11d274d25 100644
---- a/src/runtime/runtime.h
-+++ b/src/runtime/runtime.h
-@@ -120,7 +120,6 @@ namespace internal {
-   F(NotifyStubFailure, 0, 1)              \
-   F(NotifyDeoptimized, 0, 1)              \
-   F(CompileForOnStackReplacement, 1, 1)   \
--  F(TryInstallOptimizedCode, 1, 1)        \
-   F(ResolvePossiblyDirectEval, 6, 1)      \
-   F(InstantiateAsmJs, 4, 1)
- 
--- 
-2.15.1
-