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

Is V8 the best JS engine today? #72

Closed
cdnsteve opened this issue Dec 4, 2014 · 2 comments
Closed

Is V8 the best JS engine today? #72

cdnsteve opened this issue Dec 4, 2014 · 2 comments

Comments

@cdnsteve
Copy link

cdnsteve commented Dec 4, 2014

Just wondering if SpiderMoney has been given any review recently as a replacement to V8?

https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey
http://www.theregister.co.uk/2014/10/28/mozilla_claims_fastest_javascript_engine_beats_googles_v8_and/

Other than from a pure performance perspective my thinking is Mozilla's is a better fit with io.js than other corporate entities.
@gkatsev
Copy link

gkatsev commented Dec 4, 2014

They're all good engines. SpiderMonkey is definitely in some aspects while v8 is faster in others. JSC is probably faster in some other aspects.

I doubt that iojs will switch away from v8 because of how tightly integrated it is until, and if, we break it into pieces ( #9 ) but that is off the table for now because it is a major rewrite and requires a lot of time that no one has currently.

@chrisdickinson
Copy link
Contributor

It would be, at present, prohibitively time-consuming to switch Node over to a different JavaScript engine -- though it could be approached as a separate project. For more background, there's a bunch of discussion over here.

@nfriedly nfriedly mentioned this issue Nov 16, 2015
Closed
@aotenko aotenko mentioned this issue Nov 30, 2015
Closed
@skenqbx skenqbx mentioned this issue Jul 14, 2016
Closed
@LittleHelicase LittleHelicase mentioned this issue Dec 9, 2016
Closed
boingoing pushed a commit to boingoing/node that referenced this issue Apr 6, 2017
@jasongin
napi: Error handling tests and fixes (V8) (nodejs#72)
7f76b60 
 - Update tests for EH API changes
 - Update api helpers for EH API changes
 - Fix bugs in EH implementation discovered via testing
@joyeecheung joyeecheung mentioned this issue Feb 20, 2020
Closed
@snyk-bot snyk-bot mentioned this issue May 5, 2020
Open
@wmertens wmertens mentioned this issue Jun 19, 2020
Closed
@silverwind silverwind mentioned this issue Aug 20, 2020
Closed
targos added a commit to targos/node that referenced this issue Apr 17, 2021
@targos
deps: V8: cherry-pick 254c7945eea2
a08507e 
Original commit message:

    Merged: [deoptimizer] Fix bug in OptimizedFrame::Summarize

    Revision: 3353a7d0b017146d543434be4036a81aaf7d25ae

    BUG=chromium:1182647
    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    R=​bmeurer@chromium.org

    (cherry picked from commit c0c96b768a7d3463b11403874549e6496529740d)

    Change-Id: I86abd6a3f34169be5f99aa9f54bb7bb3706fa85a
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2780300
    Reviewed-by: Georg Neis <neis@chromium.org>
    Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
    Commit-Queue: Georg Neis <neis@chromium.org>
    Cr-Original-Commit-Position: refs/branch-heads/8.9@{nodejs#49}
    Cr-Original-Branched-From: 16b9bbbd581c25391981aa03180b76aa60463a3e-refs/heads/8.9.255@{#1}
    Cr-Original-Branched-From: d16a2a688498bd1c3e6a49edb25d8c4ca56232dc-refs/heads/master@{#72039}
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794427
    Reviewed-by: Victor-Gabriel Savu <vsavu@google.com>
    Commit-Queue: Artem Sumaneev <asumaneev@google.com>
    Cr-Commit-Position: refs/branch-heads/8.6@{nodejs#72}
    Cr-Branched-From: a64aed2333abf49e494d2a5ce24bbd14fff19f60-refs/heads/8.6.395@{#1}
    Cr-Branched-From: a626bc036236c9bf92ac7b87dc40c9e538b087e3-refs/heads/master@{#69472}

Refs: v8/v8@254c794
targos added a commit to targos/node that referenced this issue Apr 27, 2021
@targos
deps: V8: cherry-pick 254c7945eea2
a461185 
Original commit message:

    Merged: [deoptimizer] Fix bug in OptimizedFrame::Summarize

    Revision: 3353a7d0b017146d543434be4036a81aaf7d25ae

    BUG=chromium:1182647
    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    R=​bmeurer@chromium.org

    (cherry picked from commit c0c96b768a7d3463b11403874549e6496529740d)

    Change-Id: I86abd6a3f34169be5f99aa9f54bb7bb3706fa85a
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2780300
    Reviewed-by: Georg Neis <neis@chromium.org>
    Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
    Commit-Queue: Georg Neis <neis@chromium.org>
    Cr-Original-Commit-Position: refs/branch-heads/8.9@{nodejs#49}
    Cr-Original-Branched-From: 16b9bbbd581c25391981aa03180b76aa60463a3e-refs/heads/8.9.255@{#1}
    Cr-Original-Branched-From: d16a2a688498bd1c3e6a49edb25d8c4ca56232dc-refs/heads/master@{#72039}
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794427
    Reviewed-by: Victor-Gabriel Savu <vsavu@google.com>
    Commit-Queue: Artem Sumaneev <asumaneev@google.com>
    Cr-Commit-Position: refs/branch-heads/8.6@{nodejs#72}
    Cr-Branched-From: a64aed2333abf49e494d2a5ce24bbd14fff19f60-refs/heads/8.6.395@{#1}
    Cr-Branched-From: a626bc036236c9bf92ac7b87dc40c9e538b087e3-refs/heads/master@{#69472}

Refs: v8/v8@254c794
targos added a commit that referenced this issue Apr 30, 2021
@targos
deps: V8: cherry-pick 254c7945eea2
5dc8246 
Original commit message:

    Merged: [deoptimizer] Fix bug in OptimizedFrame::Summarize

    Revision: 3353a7d0b017146d543434be4036a81aaf7d25ae

    BUG=chromium:1182647
    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true
    R=​bmeurer@chromium.org

    (cherry picked from commit c0c96b768a7d3463b11403874549e6496529740d)

    Change-Id: I86abd6a3f34169be5f99aa9f54bb7bb3706fa85a
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2780300
    Reviewed-by: Georg Neis <neis@chromium.org>
    Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
    Commit-Queue: Georg Neis <neis@chromium.org>
    Cr-Original-Commit-Position: refs/branch-heads/8.9@{#49}
    Cr-Original-Branched-From: 16b9bbbd581c25391981aa03180b76aa60463a3e-refs/heads/8.9.255@{#1}
    Cr-Original-Branched-From: d16a2a688498bd1c3e6a49edb25d8c4ca56232dc-refs/heads/master@{#72039}
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794427
    Reviewed-by: Victor-Gabriel Savu <vsavu@google.com>
    Commit-Queue: Artem Sumaneev <asumaneev@google.com>
    Cr-Commit-Position: refs/branch-heads/8.6@{#72}
    Cr-Branched-From: a64aed2333abf49e494d2a5ce24bbd14fff19f60-refs/heads/8.6.395@{#1}
    Cr-Branched-From: a626bc036236c9bf92ac7b87dc40c9e538b087e3-refs/heads/master@{#69472}

Refs: v8/v8@254c794

PR-URL: #38275
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
@snyk-bot snyk-bot mentioned this issue Jun 23, 2021
Open
@dapplion dapplion mentioned this issue Aug 26, 2021
Closed
This was referenced Sep 13, 2021
Open
Open
Open
Open
Open
@snyk-bot snyk-bot mentioned this issue Nov 15, 2021
Open
@snyk-bot snyk-bot mentioned this issue Dec 21, 2021
Open
@snyk-bot snyk-bot mentioned this issue Feb 1, 2022
Open
@snyk-bot snyk-bot mentioned this issue Feb 28, 2022
Open
@solebox solebox mentioned this issue May 13, 2022
Open
@enterstudio enterstudio mentioned this issue May 14, 2022
Open
@UbuntuEvangelist UbuntuEvangelist mentioned this issue May 14, 2022
Open
@s4sc s4sc mentioned this issue May 14, 2022
Open
@snyk-bot snyk-bot mentioned this issue Aug 8, 2022
Open
@pratapadi pratapadi mentioned this issue Oct 5, 2022
Open
@snyk-bot snyk-bot mentioned this issue Dec 29, 2022
Open
@snyk-bot snyk-bot mentioned this issue Jan 8, 2023
Open
@snyk-bot snyk-bot mentioned this issue Feb 13, 2023
Open
@snyk-bot snyk-bot mentioned this issue May 12, 2023
Open
@Bhanditz Bhanditz mentioned this issue Nov 26, 2023
Open
@bumplzz69 bumplzz69 mentioned this issue Nov 26, 2023
Open
@RoeeOren RoeeOren mentioned this issue Nov 27, 2023
Open
@bsmoove247 bsmoove247 mentioned this issue Nov 27, 2023
Open
@Verdinjoshua26 Verdinjoshua26 mentioned this issue Nov 27, 2023
Open
@Matthelonianxl Matthelonianxl mentioned this issue Nov 27, 2023
Open
@sirinartk sirinartk mentioned this issue Nov 28, 2023
Open
@timzwit timzwit mentioned this issue Nov 28, 2023
Open
@vaibhavb vaibhavb mentioned this issue Nov 28, 2023
Open
@awaisab172 awaisab172 mentioned this issue Nov 29, 2023
Open
@ajesse11x ajesse11x mentioned this issue Nov 29, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@chrisdickinson @gkatsev @cdnsteve