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.

Sign up for GitHub

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

Jump to bottom

deps: update V8 to 6.2 #15362

Closed
wants to merge 9 commits into from
Closed

deps: update V8 to 6.2 #15362

wants to merge 9 commits into from

Commits on Oct 18, 2017

  1. deps: update V8 to 6.2.414.32

    @targos
    targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    1654217 View commit details
    Browse the repository at this point in the history

  2. src: update NODE_MODULE_VERSION to 59 

    Major V8 updates are usually API/ABI incompatible with previous
versions. This commit adapts NODE_MODULE_VERSION for V8 6.2.

Refs: https://github.com/nodejs/CTC/blob/master/meetings/2016-09-28.md
    @targos
    targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    9c8a1ae View commit details
    Browse the repository at this point in the history

  3. test: fix message test after V8 upgrade 

    With V8 6.2 there is one line less in the promise trace.
    @targos
    targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    6678dea View commit details
    Browse the repository at this point in the history

  4. ustack: update offset identifiers

    @geek @targos
    geek authored and targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    c90c3c6 View commit details
    Browse the repository at this point in the history

  5. v8abbr: fix rename of entry frame

    @geek @targos
    geek authored and targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    17049c4 View commit details
    Browse the repository at this point in the history

  6. deps: backport b096c44 from upstream V8 

    Original commit message:

    [build] Introduce an embedder version string

    Sometimes, the embedder might want to merge a fix to an abandoned branch
    or to a supported branch but the fix is not relevant to Chromium.
    This adds a new version string that the embedder can set at compile time
    and that will be appended to the official V8 version.
    The separator must be provided in the string. For instance, to have a
    full version string like "6.0.287.53-emb.1", the embedder must set
    V8_EMBEDDER_STRING to "-emb.1".

    Related Node.js issue: nodejs#9754

    BUG=v8:5740
    R=machenbach@chromium.org

    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: Ifa2d9bd213795e6d54886436f8c3787ac6162823
    Reviewed-on: https://chromium-review.googlesource.com/690475
    Reviewed-by: Michael Achenbach <machenbach@chromium.org>
    Reviewed-by: Yang Guo <yangguo@chromium.org>
    Commit-Queue: Michaël Zasso <mic.besace@gmail.com>
    Cr-Commit-Position: refs/heads/master@{nodejs#48301}

Refs: v8/v8@b096c44
PR-URL: nodejs#15785
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
    @targos
    targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    7f90060 View commit details
    Browse the repository at this point in the history

  7. deps: cherry-pick 9b21865822243 from V8 upstream 

    Original commit message:

    [api] Add optional data pointer to GC callbacks

    This can be useful when there may be multiple callbacks attached by
    code that's not directly tied to a single isolate, e.g. working
    on a per-context basis.

    This also allows rephrasing the global non-isolate APIs in terms
    of this new API, rather than working around it inside `src/heap`.

    TBR=hpayer@chromium.org

    Bug:
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: I2e490ec40d1a34ea812f25f41ef9741d2116d965
    Reviewed-on: https://chromium-review.googlesource.com/647548
    Reviewed-by: Yang Guo <yangguo@chromium.org>
    Reviewed-by: Adam Klein <adamk@chromium.org>
    Commit-Queue: Yang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{nodejs#47923}

PR-URL: nodejs#15391
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
    @addaleax @targos
    addaleax authored and targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    91b6349 View commit details
    Browse the repository at this point in the history

  8. deps: v8: fix potential segfault in profiler 

    This change fixes a potential segfault in the sampling heap profiler.
This landed as part of a larger change upstream [1]. This is the minimal
backport that avoids the segfault.

[1]: https://git.io/vdTYL

PR-URL: nodejs#15498
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
    @ofrobots @targos
    ofrobots authored and targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    eb881ff View commit details
    Browse the repository at this point in the history

  9. deps: backport 0f1dfae from V8 upstream 

    Original commit message:

    avoid constructor inheritance due to compilation issues

    Constructor inheritance of a templated constructor is causing compilation issues for node.js:

    https: //github.com/nodejs/pull/15362#issue-257007421
    Change-Id: I7d099ff5a1a2fd5b19c11112ddef8fe824e509f7
    Reviewed-on: https://chromium-review.googlesource.com/707008
    Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
    Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
    Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#48445}
    @tebbi @targos
    tebbi authored and targos committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    ec0edac View commit details
    Browse the repository at this point in the history