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

Issue 16226020: MIPS: Turn off allocation site info for crankshafted array constructor calls. (Closed)

Created:
7 years, 6 months ago by kilvadyb
Modified:
7 years, 6 months ago
Reviewers:
kisg, Paul Lind, palfia, danno
CC:
v8-dev
Visibility:
Public.

Description

MIPS: Turn off allocation site info for crankshafted array constructor calls. Port r14934 (dbddd7e) Original commit message: Once we crankshaft a method, we should turn off allocation site info for constructed arrays. Additionally, the semantics for doing this were awkward because the constructed array code stubs get an AllocationSiteMode as a minor key, but it's used as a permission to determine the final mode locally based on ElementsKind. I refactored this to a simpler boolean for override or local control. BUG=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -3 lines) Patch
M src/mips/code-stubs-mips.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M src/mips/lithium-codegen-mips.cc View 1 chunk +6 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Paul Lind
lgtm
7 years, 6 months ago (2013-06-04 16:15:14 UTC) #1
palfia
7 years, 6 months ago (2013-06-04 20:18:25 UTC) #2
Committed as r14943.

Powered by Google App Engine
This is Rietveld 408576698