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 13.0 #55014

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

deps: update V8 to 13.0 #55014

wants to merge 14 commits into from

Commits on Sep 19, 2024

  1. deps: update V8 to 13.0.245.3

    targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    3e12d70 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    b767d5b View commit details
    Browse the repository at this point in the history
  3. src: update NODE_MODULE_VERSION to 132

    Major V8 updates are usually API/ABI incompatible with previous
    versions. This commit adapts NODE_MODULE_VERSION for V8 13.0.
    
    Refs: https://github.com/nodejs/CTC/blob/master/meetings/2016-09-28.md
    targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    c4fa221 View commit details
    Browse the repository at this point in the history
  4. deps: always define V8_EXPORT_PRIVATE as no-op

    dllexport introduces issues when compiling with MSVC.
    
    PR-URL: nodejs#47251
    Reviewed-By: Yagiz Nizipli <[email protected]>
    Reviewed-By: Jiawen Geng <[email protected]>
    Reviewed-By: Rafael Gonzaga <[email protected]>
    Reviewed-By: Richard Lau <[email protected]>
    targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    67d7b10 View commit details
    Browse the repository at this point in the history
  5. deps: disable V8 concurrent sparkplug compilation

    It introduces process hangs on some platforms because Node.js doesn't
    tear down V8 correctly.
    Disable it while we work on a solution.
    
    Refs: nodejs#47297
    Refs: https://bugs.chromium.org/p/v8/issues/detail?id=13902
    PR-URL: nodejs#47450
    Reviewed-By: Richard Lau <[email protected]>
    Reviewed-By: Yagiz Nizipli <[email protected]>
    Reviewed-By: Michael Dawson <[email protected]>
    Reviewed-By: Rich Trott <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    cd0597f View commit details
    Browse the repository at this point in the history
  6. deps: patch V8 to avoid duplicated zlib symbol

    PR-URL: nodejs#54077
    Reviewed-By: Jiawen Geng <[email protected]>
    Reviewed-By: Richard Lau <[email protected]>
    Reviewed-By: Joyee Cheung <[email protected]>
    Reviewed-By: Marco Ippolito <[email protected]>
    targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    c60bcd3 View commit details
    Browse the repository at this point in the history
  7. deps: patch V8 to support compilation with MSVC

    Co-Authored-By: Michaël Zasso <[email protected]>
    PR-URL: nodejs#54536
    Reviewed-By: Antoine du Hamel <[email protected]>
    Reviewed-By: Jiawen Geng <[email protected]>
    Reviewed-By: Marco Ippolito <[email protected]>
    Reviewed-By: Richard Lau <[email protected]>
    StefanStojanovic and targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    865ae54 View commit details
    Browse the repository at this point in the history
  8. deps: fix FP16 bitcasts.h

    PR-URL: nodejs#53134
    Refs: nodejs#52809
    Reviewed-By: Yagiz Nizipli <[email protected]>
    Reviewed-By: Michaël Zasso <[email protected]>
    Reviewed-By: James M Snell <[email protected]>
    StefanStojanovic authored and targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    e364eea View commit details
    Browse the repository at this point in the history
  9. deps: always define V8_NODISCARD as no-op

    It's causing compiler errors with some classes on Xcode 11
    and the attribute should have no runtime effect.
    
    PR-URL: nodejs#54077
    Reviewed-By: Jiawen Geng <[email protected]>
    Reviewed-By: Richard Lau <[email protected]>
    Reviewed-By: Joyee Cheung <[email protected]>
    Reviewed-By: Marco Ippolito <[email protected]>
    targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    c7e9059 View commit details
    Browse the repository at this point in the history
  10. deps: patch V8 to support older Clang versions

    PR-URL: nodejs#54536
    Reviewed-By: Antoine du Hamel <[email protected]>
    Reviewed-By: Jiawen Geng <[email protected]>
    Reviewed-By: Marco Ippolito <[email protected]>
    Reviewed-By: Richard Lau <[email protected]>
    targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    cb71fac View commit details
    Browse the repository at this point in the history
  11. Configuration menu
    Copy the full SHA
    e3d7e02 View commit details
    Browse the repository at this point in the history
  12. build: remove support for ppc 32-bit

    V8 removed support for it.
    
    Refs: v8/v8@6437539
    targos committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    b0d435c View commit details
    Browse the repository at this point in the history
  13. Configuration menu
    Copy the full SHA
    8913cec View commit details
    Browse the repository at this point in the history
  14. Configuration menu
    Copy the full SHA
    d48a069 View commit details
    Browse the repository at this point in the history