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 5.4 #8317

Merged
merged 9 commits into from
Sep 22, 2016
Merged

deps: update V8 to 5.4 #8317

merged 9 commits into from
Sep 22, 2016

Commits on Sep 22, 2016

  1. deps: update V8 to 5.4.500.27 

    Pick up latest commit from the 5.4-lkgr branch.

deps: edit V8 gitignore to allow trace event copy
deps: update V8 trace event to 315bf1e2d45be7d53346c31cfcc37424a32c30c8
deps: edit V8 gitignore to allow gtest_prod.h copy
deps: update V8 gtest to 6f8a66431cb592dad629028a50b3dd418a408c87

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
    @targos
    targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    ec02b81 View commit details
    Browse the repository at this point in the history

  2. build: update V8 gypfile paths 

    The location of various gypfiles has changed in V8 5.2.

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
    @targos
    targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    fbfc15c View commit details
    Browse the repository at this point in the history

  3. build: define icu_use_data_file_flag 

    v8.gyp expects this to be defined by the embedder

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
    @ofrobots @targos
    ofrobots authored and targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    785506a View commit details
    Browse the repository at this point in the history

  4. build: use libc++ on OSX 

    V8 now depends on C++11 runtime features. On OSX this requires us to
link against the libc++ library rather than the deprecated default
that is provided with -mmacosx-version-min=10.7.

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
    @ofrobots @targos
    ofrobots authored and targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    7292a1e View commit details
    Browse the repository at this point in the history

  5. tools: do not disable ICU's transliteration 

    V8 needs it for case conversion.
Ref: https://codereview.chromium.org/1812673005

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
    @targos
    targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    2187bd3 View commit details
    Browse the repository at this point in the history

  6. build: fix mkpeephole configuration 

    mkpeephole is a binary that gets generated and run at V8 build time.
On cross-compilation builds separate build toolchains are required.
Similar to want_separate_host_toolset we default to disabled.

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
    @ofrobots @targos
    ofrobots authored and targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    19c9567 View commit details
    Browse the repository at this point in the history

  7. build: do not clean V8 gtest directory 

    There is a file in there required to build V8. The rest is being ignored
by V8's .gitignore anyway.

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
    @targos
    targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    98241f4 View commit details
    Browse the repository at this point in the history

  8. src: no longer need to use std::tr1:: 

    The use of libc++ is now forced on OSX.

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
    @targos
    targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    75242a2 View commit details
    Browse the repository at this point in the history

  9. deps: cherry-pick workaround for clang-3.4 ICE 

    Ref: nodejs#8343
Fixes: nodejs#8323

PR-URL: nodejs#8317
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
    @targos
    targos committed Sep 22, 2016
    Configuration menu
    Copy the full SHA
    91b4094 View commit details
    Browse the repository at this point in the history