Skip to content

[v8.x] deps: V8: backport d520ebb#27358

Closed
targos wants to merge 1 commit into
nodejs:v8.x-stagingfrom
targos:fix-22810
Closed

[v8.x] deps: V8: backport d520ebb#27358
targos wants to merge 1 commit into
nodejs:v8.x-stagingfrom
targos:fix-22810

Conversation

@targos

@targos targos commented Apr 23, 2019

Copy link
Copy Markdown
Member

Original commit message:

[turbofan] Fix NumberFloor typing.

Bug: chromium:841117
Change-Id: I1e83dfc82f87d0b49d3cca96290ae1d738e37d20
Reviewed-on: https://chromium-review.googlesource.com/1051228
Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
Commit-Queue: Jaroslav Sevcik <jarin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#53083}

Refs: v8/v8@d520ebb
Fixes: #22810

Original commit message:

    [turbofan] Fix NumberFloor typing.

    Bug: chromium:841117
    Change-Id: I1e83dfc82f87d0b49d3cca96290ae1d738e37d20
    Reviewed-on: https://chromium-review.googlesource.com/1051228
    Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
    Commit-Queue: Jaroslav Sevcik <jarin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{nodejs#53083}

Refs: v8/v8@d520ebb
Fixes: nodejs#22810
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot nodejs-github-bot added v8 engine Issues and PRs related to the V8 dependency. v8.x labels Apr 23, 2019
@targos targos changed the title deps: V8: backport d520ebb [v8.x] deps: V8: backport d520ebb Apr 23, 2019
@targos

targos commented Apr 23, 2019

Copy link
Copy Markdown
Member Author

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@sam-github sam-github closed this Jun 5, 2019
@targos

targos commented Jun 5, 2019

Copy link
Copy Markdown
Member Author

@sam-github why did you close this?

@BridgeAR BridgeAR reopened this Jun 5, 2019
@sam-github

Copy link
Copy Markdown
Contributor

@targos I closed it by mistake, sorry!

@targos

targos commented Jun 11, 2019

Copy link
Copy Markdown
Member Author

@nodejs/backporters

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

nodejs-github-bot commented Sep 19, 2019

Copy link
Copy Markdown
Collaborator

BethGriggs pushed a commit that referenced this pull request Sep 19, 2019
Original commit message:

    [turbofan] Fix NumberFloor typing.

    Bug: chromium:841117
    Change-Id: I1e83dfc82f87d0b49d3cca96290ae1d738e37d20
    Reviewed-on: https://chromium-review.googlesource.com/1051228
    Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
    Commit-Queue: Jaroslav Sevcik <jarin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#53083}

Refs: v8/v8@d520ebb
Fixes: #22810

PR-URL: #27358
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Refael Ackermann (רפאל פלחי) <refack@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
@BethGriggs

Copy link
Copy Markdown
Member

Landed on v8.x-staging

@BethGriggs BethGriggs closed this Sep 19, 2019
@targos targos deleted the fix-22810 branch September 19, 2019 13:47
@BethGriggs BethGriggs mentioned this pull request Sep 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants