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 10.8 #45230

Closed
wants to merge 11 commits into from
Closed

deps: update V8 to 10.8 #45230

wants to merge 11 commits into from

Commits on Nov 18, 2022

  1. deps: update V8 to 10.8.168.20

    targos committed Nov 18, 2022
    Configuration menu
    Copy the full SHA
    d71a4ab View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    c82dad1 View commit details
    Browse the repository at this point in the history
  3. 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 Nov 18, 2022
    Configuration menu
    Copy the full SHA
    ec06a49 View commit details
    Browse the repository at this point in the history
  4. deps: V8: fix v8-cppgc.h for MSVC

    Refs: https://bugs.chromium.org/p/v8/issues/detail?id=12661
    Refs: nodejs#42375
    
    PR-URL: nodejs#42657
    Reviewed-By: Darshan Sen <raisinten@gmail.com>
    Reviewed-By: Richard Lau <rlau@redhat.com>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: Michael Dawson <midawson@redhat.com>
    gengjiawen authored and targos committed Nov 18, 2022
    Configuration menu
    Copy the full SHA
    4232def View commit details
    Browse the repository at this point in the history
  5. deps: silence irrelevant V8 warning

    PR-URL: nodejs#44741
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    targos committed Nov 18, 2022
    Configuration menu
    Copy the full SHA
    05914ad View commit details
    Browse the repository at this point in the history
  6. deps: fix V8 build on Windows with MSVC

    PR-URL: nodejs#44741
    Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    targos committed Nov 18, 2022
    Configuration menu
    Copy the full SHA
    a5389a7 View commit details
    Browse the repository at this point in the history
  7. deps: V8: cherry-pick 92a7385171bb

    Original commit message:
    
        [heap] Fix 32bit msvc builds
    
        Size of ActiveSystemPages is 8 bytes even on 32bit builds, thus
        forcing 8 bytes alignment for MemoryChunk.
    
        Change-Id: I5ca1e18329d6e68a8b6811c3c27cb224c765cb63
        Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3966953
        Commit-Queue: Omer Katz <omerkatz@chromium.org>
        Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
        Cr-Commit-Position: refs/heads/main@{#83845}
    
    Refs: v8/v8@92a7385
    targos committed Nov 18, 2022
    Configuration menu
    Copy the full SHA
    d5c5217 View commit details
    Browse the repository at this point in the history
  8. deps: V8: cherry-pick f1c888e7093e

    Original commit message:
    
        [base] Fix SmartOS build
    
        That platform doesn't have `malloc_usable_size`.
    
        Refs: nodejs/node-v8#239
        Change-Id: I011dd8449d02b27219a32cba00132cd068069f50
        Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4026402
        Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
        Commit-Queue: Michaël Zasso <mic.besace@gmail.com>
        Cr-Commit-Position: refs/heads/main@{#84256}
    
    Refs: v8/v8@f1c888e
    targos committed Nov 18, 2022
    Configuration menu
    Copy the full SHA
    4e40d93 View commit details
    Browse the repository at this point in the history
  9. deps: V8: cherry-pick 9df5ef70ff18

    Original commit message:
    
        Add an `v8::ArrayBuffer::WasDetached` method to the C++ API
    
        V8's C++ API does not give a way to tell whether an ArrayBuffer has
        been detached from the `v8::ArrayBuffer` class. In fact, as far as can
        be told from the C++ API without running JS code, detached
        ArrayBuffers behave the same as zero-sized ArrayBuffers and there is
        no way to observe the difference. However, this difference can be
        observed in JS because constructing a TypedArray from a detached
        ArrayBuffer will throw.
    
        This change adds a `WasDetached` method to the `v8::ArrayBuffer` class
        to give embedders access to this information without having to run JS
        code.
    
        Bug: v8:13159
        Change-Id: I2bb1e380cee1cecd31f6d48ec3d9f28c03a8a673
        Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3810345
        Commit-Queue: Toon Verwaest <verwaest@chromium.org>
        Reviewed-by: Toon Verwaest <verwaest@chromium.org>
        Cr-Commit-Position: refs/heads/main@{#83963}
    
    Refs: v8/v8@9df5ef7
    PR-URL: nodejs#45474
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
    Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
    Reviewed-By: Rich Trott <rtrott@gmail.com>
    Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    anonrig authored and targos committed Nov 18, 2022
    Configuration menu
    Copy the full SHA
    7b1fa2f View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    dddea32 View commit details
    Browse the repository at this point in the history
  11. test: fix test-trace-gc-flag

    tony-go authored and targos committed Nov 18, 2022
    Configuration menu
    Copy the full SHA
    bbf9372 View commit details
    Browse the repository at this point in the history