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

Issue 363293004: ARM64: improve Adr patching (Closed)

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

Description

ARM64: improve Adr patching R=ulan@chromium.org, bmeurer@chromium.org BUG=

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -59 lines) Patch
M src/arm64/assembler-arm64.h View 1 chunk +3 lines, -6 lines 0 comments Download
M src/arm64/assembler-arm64.cc View 2 chunks +9 lines, -46 lines 3 comments Download
M src/arm64/instructions-arm64.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M src/arm64/macro-assembler-arm64.cc View 3 chunks +3 lines, -5 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
vincent.belliard
Reduce the patch size for Adr and remove the add instruction in the most found ...
6 years, 5 months ago (2014-07-03 10:20:30 UTC) #1
ulan
lgtm https://codereview.chromium.org/363293004/diff/1/src/arm64/assembler-arm64.cc File src/arm64/assembler-arm64.cc (left): https://codereview.chromium.org/363293004/diff/1/src/arm64/assembler-arm64.cc#oldcode3115 src/arm64/assembler-arm64.cc:3115: // add rd, rd, scratch Why not remove ...
6 years, 5 months ago (2014-07-03 10:36:21 UTC) #2
vincent.belliard
https://codereview.chromium.org/363293004/diff/1/src/arm64/assembler-arm64.cc File src/arm64/assembler-arm64.cc (left): https://codereview.chromium.org/363293004/diff/1/src/arm64/assembler-arm64.cc#oldcode3115 src/arm64/assembler-arm64.cc:3115: // add rd, rd, scratch The add has a ...
6 years, 5 months ago (2014-07-03 10:46:06 UTC) #3
ulan
https://codereview.chromium.org/363293004/diff/1/src/arm64/assembler-arm64.cc File src/arm64/assembler-arm64.cc (left): https://codereview.chromium.org/363293004/diff/1/src/arm64/assembler-arm64.cc#oldcode3115 src/arm64/assembler-arm64.cc:3115: // add rd, rd, scratch On 2014/07/03 10:46:06, vincent.belliard.arm ...
6 years, 5 months ago (2014-07-03 11:01:48 UTC) #4
vincent.belliard
6 years, 5 months ago (2014-07-03 13:20:16 UTC) #5
committed r22198

Powered by Google App Engine
This is Rietveld 408576698