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

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 10 months ago by Michael Starzinger
Modified:
2 years, 10 months ago
Reviewers:
danno
CC:
v8-dev
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) Patch
M src/compiler.cc View 2 chunks +7 lines, -3 lines 0 comments Download
M src/ic-inl.h View 2 chunks +6 lines, -2 lines 0 comments Download
M src/mark-compact.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/mark-compact.cc View 1 chunk +14 lines, -0 lines 0 comments Download
M src/objects.h View 1 chunk +3 lines, -3 lines 0 comments Download
M src/objects.cc View 4 chunks +6 lines, -8 lines 0 comments Download
M src/version.cc View 1 chunk +1 line, -1 line 0 comments Download
M src/x64/lithium-codegen-x64.cc View 1 chunk +9 lines, -3 lines 0 comments Download
M test/cctest/test-api.cc View 1 chunk +10 lines, -0 lines 0 comments Download
M test/cctest/test-heap.cc View 2 chunks +61 lines, -0 lines 0 comments Download
A + test/mjsunit/regress/regress-2326.js View 1 chunk +22 lines, -15 lines 0 comments Download
A + test/mjsunit/regress/regress-crbug-150729.js View 1 chunk +9 lines, -7 lines 0 comments Download
Trybot results:
Project "None" does not have a commit queue.

Messages

Total messages: 2 (0 generated)
Michael Starzinger
2 years, 10 months ago (2012-10-12 13:59:03 UTC) #1
danno
2 years, 10 months ago (2012-10-12 14:01:58 UTC) #2
lgtm
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld c33a7a4