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

Issue 9323002: ARM microoptimizations to megamorphic stub probing. (Closed)

Created:
8 years, 10 months ago by Erik Corry
Modified:
8 years, 10 months ago
Reviewers:
Michael Starzinger
CC:
v8-dev
Visibility:
Public.

Description

ARM microoptimizations to megamorphic stub probing. Committed: https://code.google.com/p/v8/source/detail?r=10591

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -18 lines) Patch
M src/arm/stub-cache-arm.cc View 3 chunks +34 lines, -14 lines 0 comments Download
M src/objects.h View 2 chunks +3 lines, -2 lines 1 comment Download
M src/stub-cache.h View 1 chunk +4 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Erik Corry
8 years, 10 months ago (2012-02-02 12:42:10 UTC) #1
Erik Corry
Changes this: 0x43210174 e308c202 movw ip, #33282 0x43210178 e34fcfff movt ip, #65535 0x4321017c e023300c eor ...
8 years, 10 months ago (2012-02-02 12:45:19 UTC) #2
Michael Starzinger
8 years, 10 months ago (2012-02-02 13:14:19 UTC) #3
LGTM (if comment is addressed).

https://chromiumcodereview.appspot.com/9323002/diff/1/src/objects.h
File src/objects.h (right):

https://chromiumcodereview.appspot.com/9323002/diff/1/src/objects.h#newcode4376
src/objects.h:4376: // This constant should be encodable in an ARM instruction.
I think we need to have a static-assert at the usage site in stub-cache-arm.cc
where this is relevant. Otherwise that promise will be broken soon.

Powered by Google App Engine
This is Rietveld 408576698