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

Issue 305963010: ARM64: Restructure the L1 deopt jump table. (Closed)

Created:
6 years, 6 months ago by jbramley
Modified:
6 years, 6 months ago
CC:
v8-dev
Visibility:
Public.

Description

ARM64: Restructure the L1 deopt jump table. This restructures the L1 deopt jump table so that the base address of the L2 table is only loaded once. This significantly reduces the size of the generated code because only one big immediate needs to be loaded. The total size of all L1 deopt tables generated during Octane is almost halved in size, from about 1105kB to 584kB. BUG= R=ulan@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=21608

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+67 lines, -36 lines) Patch
M src/arm64/lithium-codegen-arm64.cc View 1 chunk +67 lines, -36 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jbramley
6 years, 6 months ago (2014-06-02 12:16:49 UTC) #1
ulan
lgtm
6 years, 6 months ago (2014-06-02 13:10:24 UTC) #2
jbramley
6 years, 6 months ago (2014-06-02 13:12:23 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r21608 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698