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

Issue 209923002: ARM64: optimize call immediate (Closed)

Created:
6 years, 9 months ago by vincent.belliard
Modified:
6 years, 9 months ago
Reviewers:
Benedikt Meurer, ulan
CC:
v8-dev
Visibility:
Public.

Description

ARM64: optimize call immediate BUG= R=ulan@chromium.org, bmeurer@chromium.org

Patch Set 1 #

Total comments: 2

Patch Set 2 : review fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -19 lines) Patch
M src/arm64/assembler-arm64.h View 1 1 chunk +7 lines, -8 lines 0 comments Download
M src/arm64/assembler-arm64-inl.h View 1 2 chunks +8 lines, -10 lines 0 comments Download
M src/arm64/macro-assembler-arm64.cc View 1 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
vincent.belliard
6 years, 9 months ago (2014-03-24 09:37:20 UTC) #1
ulan
https://codereview.chromium.org/209923002/diff/1/src/arm64/macro-assembler-arm64.cc File src/arm64/macro-assembler-arm64.cc (right): https://codereview.chromium.org/209923002/diff/1/src/arm64/macro-assembler-arm64.cc#newcode2027 src/arm64/macro-assembler-arm64.cc:2027: // Addresses are 48 bits so we never need ...
6 years, 9 months ago (2014-03-24 12:45:53 UTC) #2
vincent.belliard
Address Ulan comments
6 years, 9 months ago (2014-03-24 15:41:57 UTC) #3
ulan
6 years, 9 months ago (2014-03-24 15:44:58 UTC) #4
lgtm, thanks for fixing it!

Powered by Google App Engine
This is Rietveld 408576698