deps: backport 819b40a from V8 upstream (v4.x) #3938
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original commit message:
Use baseline code to compute message locations.
This switches Isolate::ComputeLocation to use baseline code when
computing message locations. This unifies locations between optimized
and non-optimized code by always going through the FrameSummary for
location computation.
R=bmeurer@chromium.org
TEST=message/regress/regress-4266
BUG=v8:4266
LOG=n
Review URL: https://codereview.chromium.org/1331603002
Cr-Commit-Position: refs/heads/master@{nodejs#30635}
Fixes: nodejs#3934
targos
added
v8 engine
|
Rubber stamp LGTM |
|
LGTM |
|
LGTM |
|
Updated here like #3937. V8 tests pass on the 4.5 branch. |
|
LGTM |
targos
added a commit
that referenced
this pull request
Nov 25, 2015
Original commit message:
Use baseline code to compute message locations.
This switches Isolate::ComputeLocation to use baseline code when
computing message locations. This unifies locations between optimized
and non-optimized code by always going through the FrameSummary for
location computation.
R=bmeurer@chromium.org
TEST=message/regress/regress-4266
BUG=v8:4266
LOG=n
Review URL: https://codereview.chromium.org/1331603002
Cr-Commit-Position: refs/heads/master@{#30635}
Fixes: #3934
PR-URL: #3938
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
|
landed on |
targos
added a commit
that referenced
this pull request
Dec 4, 2015
Original commit message:
Use baseline code to compute message locations.
This switches Isolate::ComputeLocation to use baseline code when
computing message locations. This unifies locations between optimized
and non-optimized code by always going through the FrameSummary for
location computation.
R=bmeurer@chromium.org
TEST=message/regress/regress-4266
BUG=v8:4266
LOG=n
Review URL: https://codereview.chromium.org/1331603002
Cr-Commit-Position: refs/heads/master@{#30635}
Fixes: #3934
PR-URL: #3938
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Closed
targos
added a commit
that referenced
this pull request
Dec 17, 2015
Original commit message:
Use baseline code to compute message locations.
This switches Isolate::ComputeLocation to use baseline code when
computing message locations. This unifies locations between optimized
and non-optimized code by always going through the FrameSummary for
location computation.
R=bmeurer@chromium.org
TEST=message/regress/regress-4266
BUG=v8:4266
LOG=n
Review URL: https://codereview.chromium.org/1331603002
Cr-Commit-Position: refs/heads/master@{#30635}
Fixes: #3934
PR-URL: #3938
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
jasnell
added a commit
that referenced
this pull request
Dec 17, 2015
Maintenance Update
Notable changes
* Roughly 78% of the commits are documentation and test
improvements
* domains:
- Fix handling of uncaught exceptions (Julien Gilli)
[#3884](#3884)
* deps:
- Upgrade to npm 2.14.12 (Kat Marchán)
[#4110](#4110)
- Backport 819b40a from V8 upstream (Michaël Zasso)
[#3938](#3938)
- Updated node LICENSE file with new npm license (Kat Marchán)
[#4110](#4110)
jasnell
added a commit
that referenced
this pull request
Dec 21, 2015
Maintenance Update
Notable changes
* Roughly 78% of the commits are documentation and test
improvements
* domains:
- Fix handling of uncaught exceptions (Julien Gilli)
[#3884](#3884)
* deps:
- Upgrade to npm 2.14.12 (Kat Marchán)
[#4110](#4110)
- Backport 819b40a from V8 upstream (Michaël Zasso)
[#3938](#3938)
- Updated node LICENSE file with new npm license (Kat Marchán)
[#4110](#4110)
targos
added a commit
that referenced
this pull request
Dec 23, 2015
Original commit message:
Use baseline code to compute message locations.
This switches Isolate::ComputeLocation to use baseline code when
computing message locations. This unifies locations between optimized
and non-optimized code by always going through the FrameSummary for
location computation.
R=bmeurer@chromium.org
TEST=message/regress/regress-4266
BUG=v8:4266
LOG=n
Review URL: https://codereview.chromium.org/1331603002
Cr-Commit-Position: refs/heads/master@{#30635}
Fixes: #3934
PR-URL: #3938
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
jasnell
added a commit
that referenced
this pull request
Dec 23, 2015
Maintenance Update
Notable changes
* Roughly 78% of the commits are documentation and test
improvements
* domains:
- Fix handling of uncaught exceptions (Julien Gilli)
[#3884](#3884)
* deps:
- Upgrade to npm 2.14.12 (Kat Marchán)
[#4110](#4110)
- Backport 819b40a from V8 upstream (Michaël Zasso)
[#3938](#3938)
- Updated node LICENSE file with new npm license (Kat Marchán)
[#4110](#4110)
jasnell
added a commit
that referenced
this pull request
Dec 23, 2015
Maintenance Update
Notable changes
* Roughly 78% of the commits are documentation and test
improvements
* domains:
- Fix handling of uncaught exceptions (Julien Gilli)
[#3884](#3884)
* deps:
- Upgrade to npm 2.14.12 (Kat Marchán)
[#4110](#4110)
- Backport 819b40a from V8 upstream (Michaël Zasso)
[#3938](#3938)
- Updated node LICENSE file with new npm license (Kat Marchán)
[#4110](#4110)
scovetta
pushed a commit
to scovetta/node
that referenced
this pull request
Apr 2, 2016
Maintenance Update
Notable changes
* Roughly 78% of the commits are documentation and test
improvements
* domains:
- Fix handling of uncaught exceptions (Julien Gilli)
[nodejs#3884](nodejs#3884)
* deps:
- Upgrade to npm 2.14.12 (Kat Marchán)
[nodejs#4110](nodejs#4110)
- Backport 819b40a from V8 upstream (Michaël Zasso)
[nodejs#3938](nodejs#3938)
- Updated node LICENSE file with new npm license (Kat Marchán)
[nodejs#4110](nodejs#4110)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Original commit message:
Fixes: #3934
cc @nodejs/v8 @nodejs/lts