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: update V8 to 5.9 #13263

Closed
wants to merge 10 commits into from
Closed

deps: update V8 to 5.9 #13263

wants to merge 10 commits into from

Commits on Jun 6, 2017

  1. deps: update V8 to 5.9.211.32

    targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    b835efa View commit details
    Browse the repository at this point in the history
  2. src: update NODE_MODULE_VERSION to 56

    Major V8 updates are usually API/ABI incompatible with previous
    versions. This commit adapts NODE_MODULE_VERSION for V8 5.9.
    
    Refs: https://github.com/nodejs/CTC/blob/master/meetings/2016-09-28.md
    targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    abc16da View commit details
    Browse the repository at this point in the history
  3. deps: limit regress/regress-crbug-514081 v8 test

    regress/regress-crbug-514081 allocates a 2G block of memory
    and if there  are multiple variants running at the
    same time this can lead to crashes, OOM kills or
    the OS failing to allocate memory.  This patch
    limits us to running a single variant of the test
    
    Fixes: nodejs#6340
    PR-URL: nodejs#6678
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Fedor Indutny <fedor@indutny.com>
    mhdawson authored and targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    9ccfdb6 View commit details
    Browse the repository at this point in the history
  4. deps: run memory hungry V8 test in exclusive mode

    es6/typedarray-construct-offset-not-smi allocates a 2G block of memory
    and if there  are multiple variants running at the same time this can
    lead to crashes, OOM kills or the OS failing to allocate memory.
    This patch limits us to running a single variant of the test.
    
    Refs: nodejs#6678
    targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    ad4892e View commit details
    Browse the repository at this point in the history
  5. deps: add missing include to V8 i18n.cc

    This is required for ICU 59.1.
    targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    e64561a View commit details
    Browse the repository at this point in the history
  6. deps: fix addons compilation with VS2013

    VS2013 does not support defaulting move constructor and assignment
    operator. This adds explicit definitions of those methods for two
    classes.
    This fix is required because we still support building addons with
    VS2013 and the incompatibility is in v8.h.
    
    Fixes: nodejs/node-v8#4
    bzoz authored and targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    f5e6af4 View commit details
    Browse the repository at this point in the history
  7. v8: fix stack overflow in recursive method

    HGlobalValueNumberingPhase::CollectSideEffectsOnPathsToDominatedBlock()
    used to self-recurse before this commit, causing stack overflows on
    systems with small stack sizes.  Make it non-recursive by storing
    intermediate results in a heap-allocated list.
    
    Fixes: nodejs#11991
    PR-URL: nodejs#12460
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Yang Guo <yangguo@chromium.org>
    bnoordhuis authored and targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    52bbe00 View commit details
    Browse the repository at this point in the history
  8. deps: cherry-pick bfae9db from upstream v8

    Original commit message:
    
        Update postmortem metadata generator.
    
        Add PropertyDetails::AttributesField +
        PropertyDetails::LocationField.
    
        Review-Url: https://codereview.chromium.org/2842843004
        Cr-Commit-Position: refs/heads/master@{nodejs#44889}
    
    PR-URL: nodejs#12722
    Refs: nodejs/llnode#81
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    bnoordhuis authored and targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    de93811 View commit details
    Browse the repository at this point in the history
  9. deps: cherry-pick f5fad6d from upstream v8

    Original commit message:
    
        This commit adds a getter for the private is_verbose_ member.
        The use case for this comes from Node.js where the ability to avoid
        calling FatalException if the TryCatch is verbose would be nice
        to have.
    
        BUG=
    
        Review-Url: https://codereview.chromium.org/2840803002
        Cr-Commit-Position: refs/heads/master@{nodejs#45018}
    
    PR-URL: nodejs#12826
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    danbev authored and targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    9936eb9 View commit details
    Browse the repository at this point in the history
  10. deps: cherry-pick 6d38f89 from upstream V8

    Original commit message:
    
        [turbofan] Boost performance of Array.prototype.shift by 4x.
    
        For small arrays, it's way faster to just move the elements instead of
        doing the fairly complex and heavy-weight left-trimming. Crankshaft has
        had this optimization for small arrays already; this CL more or less
        ports this functionality to TurboFan, which yields a 4x speed-up when
        using shift on small arrays (with up to 16 elements).
    
        This should recover some of the regressions reported in the Node.js issues
    
          nodejs#12657
    
        and discovered for the syncthrough module using
    
          https://github.com/mcollina/syncthrough/blob/master/benchmarks/basic.js
    
        as benchmark.
    
        R=jarin@chromium.org
        BUG=v8:6376
    
        Review-Url: https://codereview.chromium.org/2874453002
        Cr-Commit-Position: refs/heads/master@{nodejs#45216}
    targos committed Jun 6, 2017
    Configuration menu
    Copy the full SHA
    59ac2a2 View commit details
    Browse the repository at this point in the history