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

Issue 334913004: Support external startup data in V8. (retry) (Closed)

Created:
6 years, 6 months ago by vogelheim
Modified:
6 years, 6 months ago
CC:
v8-dev, Paweł Hajdan Jr.
Visibility:
Public.

Description

Support external startup data in V8. [Re-retry of r21696 and r21739] If the embedder chooses, the 'natives' (library sources) and the precompiled startup blob can be written to files during the build process and handed over to V8 at startup. The main purpose would be to reduce the size of the compiled binary for space constrained platforms. The build-time option is off by default. Nothing should change if it's not enabled. BUG= R=jochen@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=21941

Patch Set 1 : Re-revert of r21696 and r21739. #

Patch Set 2 : Fix the issues. #

Patch Set 3 : rebase + style fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1115 lines, -270 lines) Patch
M BUILD.gn View 1 2 11 chunks +106 lines, -13 lines 0 comments Download
M build/features.gypi View 1 2 2 chunks +8 lines, -3 lines 0 comments Download
M include/v8.h View 1 2 1 chunk +18 lines, -0 lines 0 comments Download
M src/api.cc View 1 2 2 chunks +18 lines, -2 lines 0 comments Download
M src/d8.h View 2 chunks +5 lines, -1 line 0 comments Download
M src/d8.cc View 1 2 2 chunks +64 lines, -0 lines 0 comments Download
M src/flag-definitions.h View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
M src/heap.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M src/mksnapshot.cc View 1 2 5 chunks +59 lines, -12 lines 0 comments Download
M src/natives.h View 2 chunks +9 lines, -0 lines 0 comments Download
A src/natives-external.cc View 1 1 chunk +196 lines, -0 lines 0 comments Download
M src/serialize.h View 1 2 4 chunks +3 lines, -75 lines 0 comments Download
M src/serialize.cc View 1 2 3 chunks +1 line, -21 lines 0 comments Download
M src/snapshot.h View 2 chunks +7 lines, -10 lines 0 comments Download
M src/snapshot-common.cc View 1 2 3 chunks +13 lines, -53 lines 0 comments Download
A src/snapshot-external.cc View 1 chunk +140 lines, -0 lines 0 comments Download
A src/snapshot-source-sink.h View 1 chunk +88 lines, -0 lines 0 comments Download
A src/snapshot-source-sink.cc View 1 chunk +95 lines, -0 lines 0 comments Download
M test/cctest/test-serialize.cc View 1 2 3 chunks +56 lines, -39 lines 0 comments Download
A + tools/concatenate-files.py View 2 chunks +42 lines, -21 lines 0 comments Download
M tools/gyp/v8.gyp View 1 2 10 chunks +132 lines, -7 lines 0 comments Download
M tools/js2c.py View 8 chunks +52 lines, -10 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
vogelheim
6 years, 6 months ago (2014-06-13 19:43:13 UTC) #1
jochen (gone - plz use gerrit)
lgtm
6 years, 6 months ago (2014-06-13 22:36:40 UTC) #2
vogelheim
6 years, 6 months ago (2014-06-23 13:53:44 UTC) #3
Message was sent while issue was closed.
Committed patchset #3 manually as r21941 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698