Chromium Code Reviews
Help | Chromium Project | Sign in
(486)

Issue 11117020: Merged r12504, r12531, r12562, r12563, r12655 into 3.13 branch. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 6 months ago by Michael Starzinger
Modified:
1 year, 6 months ago
Reviewers:
danno
CC:
v8-dev_googlegroups.com
Visibility:
Public.

Description

Merged r12504, r12531, r12562, r12563, r12655 into 3.13 branch.

Fix caching of optimized code for OSR.

Fix casting error for receiver of interceptors.

Fix LBoundsCheck on x64 to handle (stack slot + constant) correctly

Fix missing slot recodring during clearing of CallICs.

Fix slot recording of code target patches.

R=danno@chromium.org
BUG=v8:2326, chromium:149912, 150729, chromium:144230, chromium:152615, chromium:144230
Committed: https://code.google.com/p/v8/source/detail?r=12716

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+149 lines, -42 lines) Lint Patch
M src/compiler.cc View 2 chunks +7 lines, -3 lines 0 comments ? errors Download
M src/ic-inl.h View 2 chunks +6 lines, -2 lines 0 comments ? errors Download
M src/mark-compact.h View 1 chunk +1 line, -0 lines 0 comments ? errors Download
M src/mark-compact.cc View 1 chunk +14 lines, -0 lines 0 comments ? errors Download
M src/objects.h View 1 chunk +3 lines, -3 lines 0 comments ? errors Download
M src/objects.cc View 4 chunks +6 lines, -8 lines 0 comments ? errors Download
M src/version.cc View 1 chunk +1 line, -1 line 0 comments ? errors Download
M src/x64/lithium-codegen-x64.cc View 1 chunk +9 lines, -3 lines 0 comments ? errors Download
M test/cctest/test-api.cc View 1 chunk +10 lines, -0 lines 0 comments ? errors Download
M test/cctest/test-heap.cc View 2 chunks +61 lines, -0 lines 0 comments ? errors Download
A + test/mjsunit/regress/regress-2326.js View 1 chunk +22 lines, -15 lines 0 comments ? errors Download
A + test/mjsunit/regress/regress-crbug-150729.js View 1 chunk +9 lines, -7 lines 0 comments ? errors Download
Trybot results:
Commit:

Messages

Total messages: 2
Michael Starzinger
1 year, 6 months ago #1
danno
1 year, 6 months ago #2
lgtm
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6