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

Issue 12178017: Tag stubs that rely on instance types as MEGAMORPHIC. (Closed)

Created:
7 years, 10 months ago by Toon Verwaest
Modified:
7 years, 10 months ago
Reviewers:
Jakob Kummerow
CC:
v8-dev
Visibility:
Public.

Description

Tag stubs that rely on instance types as MEGAMORPHIC. BUG=chromium:173974 Committed: https://code.google.com/p/v8/source/detail?r=13586

Patch Set 1 : #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -8 lines) Patch
M src/builtins.h View 1 chunk +1 line, -1 line 0 comments Download
M src/code-stubs.h View 1 1 chunk +6 lines, -1 line 0 comments Download
A + test/mjsunit/regress/regress-crbug-173974.js View 1 chunk +8 lines, -6 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Toon Verwaest
PTAL
7 years, 10 months ago (2013-02-04 12:59:51 UTC) #1
Jakob Kummerow
LGTM with nits. https://chromiumcodereview.appspot.com/12178017/diff/2001/src/code-stubs.h File src/code-stubs.h (right): https://chromiumcodereview.appspot.com/12178017/diff/2001/src/code-stubs.h#newcode553 src/code-stubs.h:553: // TODO(verwaest): These stubs should collect ...
7 years, 10 months ago (2013-02-04 13:04:06 UTC) #2
Toon Verwaest
7 years, 10 months ago (2013-02-04 13:11:43 UTC) #3
addressed nits.

Powered by Google App Engine
This is Rietveld 408576698