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

Issue 638423003: Split off remaining runtime functions in runtime.cc. (Closed)

Created:
6 years, 2 months ago by Yang
Modified:
6 years, 2 months ago
Reviewers:
Benedikt Meurer
CC:
v8-dev
Project:
v8
Visibility:
Public.

Description

Split off remaining runtime functions in runtime.cc. R=bmeurer@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=24533

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3256 lines, -3212 lines) Patch
M BUILD.gn View 1 chunk +5 lines, -1 line 0 comments Download
M src/macros.py View 1 chunk +1 line, -1 line 0 comments Download
M src/runtime/runtime.cc View 3 chunks +5 lines, -3210 lines 0 comments Download
A src/runtime/runtime-api.cc View 1 chunk +127 lines, -0 lines 0 comments Download
A src/runtime/runtime-array.cc View 1 chunk +1161 lines, -0 lines 0 comments Download
M src/runtime/runtime-debug.cc View 1 chunk +6 lines, -0 lines 0 comments Download
A src/runtime/runtime-internal.cc View 1 chunk +283 lines, -0 lines 0 comments Download
M src/runtime/runtime-numbers.cc View 2 chunks +16 lines, -0 lines 0 comments Download
A src/runtime/runtime-object.cc View 1 chunk +1642 lines, -0 lines 0 comments Download
M src/runtime/runtime-strings.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M tools/gyp/v8.gyp View 2 chunks +4 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Yang
6 years, 2 months ago (2014-10-09 14:44:42 UTC) #1
Benedikt Meurer
LGTM, but maybe wait 'til evening with the commit.
6 years, 2 months ago (2014-10-10 04:04:33 UTC) #2
Yang
6 years, 2 months ago (2014-10-10 15:00:09 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as 24533 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698