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 9.7 #40907

Closed
wants to merge 17 commits into from
Closed

deps: update V8 to 9.7 #40907

wants to merge 17 commits into from

Commits on Jan 18, 2022

  1. deps: update V8 to 9.7.106.18

    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    9dc4c92 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    c534c2a View commit details
    Browse the repository at this point in the history
  3. src: update NODE_MODULE_VERSION to 104

    Major V8 updates are usually API/ABI incompatible with previous
    versions. This commit adapts NODE_MODULE_VERSION for V8 9.7.
    
    Refs: https://github.com/nodejs/CTC/blob/master/meetings/2016-09-28.md
    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    16e8671 View commit details
    Browse the repository at this point in the history
  4. deps: V8: un-cherry-pick bd019bd

    Original commit message:
    
        [testrunner] delete ancient junit compatible format support
    
        Testrunner has ancient support for JUnit compatible XML output.
    
        This CL removes this old feature.
    
        R=mstarzinger@chromium.org,jgruber@chromium.org,jkummerow@chromium.org
        CC=​machenbach@chromium.org
    
        Bug: v8:8728
        Change-Id: I7e1beb011dbaec3aa1a27398a5c52abdd778eaf0
        Reviewed-on: https://chromium-review.googlesource.com/c/1430065
        Reviewed-by: Jakob Gruber <jgruber@chromium.org>
        Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
        Commit-Queue: Tamer Tas <tmrts@chromium.org>
        Cr-Commit-Position: refs/heads/master@{#59045}
    
    Refs: v8/v8@bd019bd
    
    PR-URL: nodejs#32116
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
    refack authored and targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    89b19fe View commit details
    Browse the repository at this point in the history
  5. deps: V8: patch register-arm64.h

    Fixes a compilation issue on some platforms
    
    PR-URL: nodejs#32116
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
    refack authored and targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    9fcbe41 View commit details
    Browse the repository at this point in the history
  6. deps: V8: forward declaration of Rtl*FunctionTable

    This should be semver-patch since actual invocation is version
    conditional.
    
    PR-URL: nodejs#32116
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
    refack authored and targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    209b698 View commit details
    Browse the repository at this point in the history
  7. deps: make v8.h compatible with VS2015

    There is a bug in the most recent version of VS2015 that affects v8.h
    and therefore prevents compilation of addons.
    
    Refs: https://stackoverflow.com/q/38378693
    
    PR-URL: nodejs#32116
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
    joaocgreis authored and targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    04adc8a View commit details
    Browse the repository at this point in the history
  8. deps: fix V8 build issue with inline methods

    PR-URL: nodejs#35415
    Reviewed-By: Rich Trott <rtrott@gmail.com>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
    Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
    Reviewed-By: Myles Borins <myles.borins@gmail.com>
    
    deps: patch for v8 on windows
    
    PR-URL: nodejs#40010
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    
    deps: patch v8 for vs2019 in std17
    
    PR-URL: nodejs#40060
    Reviewed-By: Richard Lau <rlau@redhat.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    
    deps: workaround debug link error on Windows
    
    PR-URL: nodejs#38807
    Refs: nodejs#38788
    Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: Ash Cripps <acripps@redhat.com>
    gengjiawen authored and targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    97d2f1f View commit details
    Browse the repository at this point in the history
  9. deps: silence irrelevant V8 warning

    PR-URL: nodejs#37587
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    Reviewed-By: Myles Borins <myles.borins@gmail.com>
    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    4b7bc93 View commit details
    Browse the repository at this point in the history
  10. deps: silence irrelevant V8 warning

    PR-URL: nodejs#38990
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
    Reviewed-By: Robert Nagy <ronagy@icloud.com>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    d143cee View commit details
    Browse the repository at this point in the history
  11. deps: disable trap handler for Windows cross-compiler

    `handler-outside-simulator.cc` uses inline assembly, which is not
    supported by MSVC.
    
    PR-URL: nodejs#40488
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    c2858ca View commit details
    Browse the repository at this point in the history
  12. deps: V8: cherry-pick 7ae0b77628f6

    Original commit message:
    
        [interpreter] Stop jump-table optimizing switch stms when spread overflows
    
        Bug: v8:12389
        Change-Id: I53c728ab0c8ba38c7dd96c7e1089f771ba44b9f0
        Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289227
        Reviewed-by: Leszek Swirski <leszeks@chromium.org>
        Commit-Queue: Leszek Swirski <leszeks@chromium.org>
        Cr-Commit-Position: refs/heads/main@{#77995}
    
    Refs: v8/v8@7ae0b77
    
    PR-URL: nodejs#40882
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    rayw000 authored and targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    740526e View commit details
    Browse the repository at this point in the history
  13. deps: V8: cherry-pick 3b6b21f595f6

    Original commit message:
    
        [liftoff] Allow bailout for missing ARMv7
    
        The bailout is there explicitly in the code, so we should allow it in
        {CheckBailoutAllowed}.
    
        R=ahaas@chromium.org
    
        Bug: v8:12527
        Change-Id: Ifd906afb5f034f05c2bf7d9a28e3ab458549e7ef
        Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3372915
        Reviewed-by: Andreas Haas <ahaas@chromium.org>
        Commit-Queue: Clemens Backes <clemensb@chromium.org>
        Cr-Commit-Position: refs/heads/main@{#78515}
    
    Refs: v8/v8@3b6b21f
    
    Fixes: nodejs#41402
    
    PR-URL: nodejs#41457
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Richard Lau <rlau@redhat.com>
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    Reviewed-By: Tobias Nießen <tniessen@tnie.de>
    Reviewed-By: Darshan Sen <raisinten@gmail.com>
    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    b361e0d View commit details
    Browse the repository at this point in the history
  14. deps: V8: cherry-pick 1cc12b278e22

    Original commit message:
    
        [ic] Add StoreOwnIC_Slow
    
        This runtime function behaves like StoreDataPropertyInLiteral, except it
        can throw, since it's also used for defining public class fields. Unlike
        the literal use case, class field can end up throwing due to field
        initializers doing things like freezing the instance.
    
        Bug: chromium:1264828
        Change-Id: I3ea4d15ad9b906c26763f022c8e22b757fa80b6c
        Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3252558
        Commit-Queue: Igor Sheludko <ishell@chromium.org>
        Auto-Submit: Shu-yu Guo <syg@chromium.org>
        Reviewed-by: Igor Sheludko <ishell@chromium.org>
        Cr-Commit-Position: refs/heads/main@{#77704}
    
    Refs: v8/v8@1cc12b2
    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    c7375c7 View commit details
    Browse the repository at this point in the history
  15. deps: V8: cherry-pick 80bbbb143c24

    Original commit message:
    
        [class] handle existing readonly properties in StoreOwnIC
    
        Previously, StoreOwnIC incorrectly reuses the [[Set]] semantics
        when initializing public literal class fields and object literals in
        certain cases (e.g. when there's no feedback).
        This was less of an issue for object literals, but with public class
        fields it's possible to define property attributes while the
        instance is still being initialized, or to encounter existing static
        "name" or "length" properties that should be readonly. This patch
        fixes it by
    
        1) Emitting code that calls into the slow stub when
           handling StoreOwnIC with existing read-only properties.
        2) Adding extra steps in StoreIC::Store to handle such stores
           properly with [[DefineOwnProperty]] semantics.
    
        Bug: v8:12421, v8:9888
        Change-Id: I6547320a1caba58c66ee1043cd3183a2de7cefef
        Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3300092
        Reviewed-by: Shu-yu Guo <syg@chromium.org>
        Reviewed-by: Toon Verwaest <verwaest@chromium.org>
        Commit-Queue: Joyee Cheung <joyee@igalia.com>
        Cr-Commit-Position: refs/heads/main@{#78659}
    
    Refs: v8/v8@80bbbb1
    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    3d849f9 View commit details
    Browse the repository at this point in the history
  16. Configuration menu
    Copy the full SHA
    5fc5353 View commit details
    Browse the repository at this point in the history
  17. deps: silence V8's warning on CompileFunction

    We are not ready to migrate yet.
    
    Refs: nodejs/node-v8#214
    targos committed Jan 18, 2022
    Configuration menu
    Copy the full SHA
    dc8f296 View commit details
    Browse the repository at this point in the history