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

Issue 535733004: Minor-key-ify remaining code stubs. (Closed)

Created:
6 years, 3 months ago by Yang
Modified:
6 years, 3 months ago
Reviewers:
mvstanton
CC:
v8-dev
Project:
v8
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+280 lines, -385 lines) Patch
M src/arm/code-stubs-arm.h View 8 chunks +58 lines, -43 lines 0 comments Download
M src/arm/code-stubs-arm.cc View 12 chunks +22 lines, -129 lines 0 comments Download
M src/arm64/code-stubs-arm64.h View 7 chunks +42 lines, -36 lines 1 comment Download
M src/arm64/code-stubs-arm64.cc View 10 chunks +26 lines, -37 lines 0 comments Download
M src/code-stubs.h View 1 chunk +1 line, -1 line 0 comments Download
M src/ia32/code-stubs-ia32.h View 6 chunks +50 lines, -37 lines 0 comments Download
M src/ia32/code-stubs-ia32.cc View 11 chunks +17 lines, -31 lines 0 comments Download
M src/x64/code-stubs-x64.h View 5 chunks +50 lines, -43 lines 0 comments Download
M src/x64/code-stubs-x64.cc View 11 chunks +14 lines, -28 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Yang
6 years, 3 months ago (2014-09-03 08:26:40 UTC) #1
mvstanton
I think there is one error on arm64. If you fix that, then: LGTM. https://codereview.chromium.org/535733004/diff/1/src/arm64/code-stubs-arm64.h ...
6 years, 3 months ago (2014-09-03 11:00:48 UTC) #2
Yang
6 years, 3 months ago (2014-09-03 11:42:11 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as 23643 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698