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

Issue 11438017: Pass IC data and arguments descriptor to IC miss runtime functions. (Closed)

Created:
8 years ago by Kevin Millikin (Google)
Modified:
8 years ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Pass IC data and arguments descriptor to IC miss runtime functions. Before, we obtained them by pattern matching backwards on the machine instructions at the call site. This previous approach becomes unwieldy when we need to use to use multiple instance call patterns. R=vegorov@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=15737

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+130 lines, -155 lines) Patch
M runtime/vm/code_generator.h View 2 chunks +9 lines, -2 lines 0 comments Download
M runtime/vm/code_generator.cc View 7 chunks +40 lines, -27 lines 0 comments Download
M runtime/vm/code_patcher.h View 2 chunks +7 lines, -8 lines 0 comments Download
M runtime/vm/code_patcher_ia32.cc View 3 chunks +11 lines, -41 lines 0 comments Download
M runtime/vm/code_patcher_ia32_test.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M runtime/vm/code_patcher_x64.cc View 3 chunks +12 lines, -41 lines 0 comments Download
M runtime/vm/code_patcher_x64_test.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M runtime/vm/debugger.cc View 2 chunks +15 lines, -14 lines 0 comments Download
M runtime/vm/deopt_instructions.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M runtime/vm/object.cc View 1 chunk +1 line, -1 line 0 comments Download
M runtime/vm/stub_code_ia32.cc View 3 chunks +15 lines, -8 lines 0 comments Download
M runtime/vm/stub_code_x64.cc View 3 chunks +14 lines, -7 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Kevin Millikin (Google)
8 years ago (2012-12-05 14:11:22 UTC) #1
Vyacheslav Egorov (Google)
8 years ago (2012-12-05 15:22:56 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698