Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: V8: cherry-pick ac0fe8ec from upstream #17695

Merged
merged 1 commit into from
Dec 22, 2017
Merged

Commits on Dec 22, 2017

  1. deps: V8: cherry-pick ac0fe8ec from upstream

    Original commit message:
      [interpreter] 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@{nodejs#48667}
    
    Ref: https://bugs.chromium.org/p/v8/issues/detail?id=6933
    Ref: v8/v8@ac0fe8e
    PR-URL: nodejs#17695
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    ofrobots committed Dec 22, 2017
    Configuration menu
    Copy the full SHA
    f454e14 View commit details
    Browse the repository at this point in the history