Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(105)

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

Created:
8 years, 2 months ago by Michael Starzinger
Modified:
8 years, 2 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

Messages

Total messages: 2 (0 generated)
Michael Starzinger
8 years, 2 months ago (2012-10-12 13:59:03 UTC) #1
danno
8 years, 2 months ago (2012-10-12 14:01:58 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698