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

Issue 9159008: MIPS: Change inlined cache of intanceof stub to use indirection through cell. (Closed)

Created:
8 years, 11 months ago by kalmard
Modified:
8 years, 11 months ago
CC:
v8-dev
Visibility:
Public.

Description

MIPS: Change inlined cache of intanceof stub to use indirection through cell. Port r10380 (46f646). Original commit message: The stub was directly patching caller's code without issuing write barrier which violated incremental marking invariants. BUG= TEST= Committed: https://code.google.com/p/v8/source/detail?r=10457

Patch Set 1 #

Total comments: 6

Patch Set 2 : Updated based on review comments. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -5 lines) Patch
M src/mips/code-stubs-mips.cc View 2 chunks +5 lines, -4 lines 0 comments Download
M src/mips/lithium-codegen-mips.cc View 1 chunk +4 lines, -1 line 0 comments Download
M src/mips/macro-assembler-mips.h View 1 chunk +4 lines, -0 lines 0 comments Download
M src/mips/macro-assembler-mips.cc View 1 1 chunk +28 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
kalmard
Should be applied to the 3.7 branch as well (merged in r10387 for ARM). Successful ...
8 years, 11 months ago (2012-01-19 12:45:46 UTC) #1
Vyacheslav Egorov (Chromium)
http://codereview.chromium.org/9159008/diff/1/src/mips/macro-assembler-mips.cc File src/mips/macro-assembler-mips.cc (right): http://codereview.chromium.org/9159008/diff/1/src/mips/macro-assembler-mips.cc#newcode4740 src/mips/macro-assembler-mips.cc:4740: Check(eq, "The instruction to patch should be a lui.", ...
8 years, 11 months ago (2012-01-20 11:50:41 UTC) #2
kalmard
Thanks for the review, the issues are fixed. http://codereview.chromium.org/9159008/diff/1/src/mips/macro-assembler-mips.cc File src/mips/macro-assembler-mips.cc (right): http://codereview.chromium.org/9159008/diff/1/src/mips/macro-assembler-mips.cc#newcode4740 src/mips/macro-assembler-mips.cc:4740: Check(eq, ...
8 years, 11 months ago (2012-01-20 12:13:32 UTC) #3
Vyacheslav Egorov (Chromium)
8 years, 11 months ago (2012-01-20 12:19:06 UTC) #4
LGTM.

Erik will land it for you.

Powered by Google App Engine
This is Rietveld 408576698