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

Issue 2790243003: Do not rewrite the VM isolate's symbol table during AOT snapshots. (Closed)

Created:
3 years, 8 months ago by rmacnak
Modified:
3 years, 8 months ago
Reviewers:
siva
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Do not rewrite the VM isolate's symbol table during AOT snapshots. This has the effect of making every program generate the same VM isolate, allowing an embedder to load different programs into different isolates. Verify the number of base objects in clustered snapshots. Remove dead code leftover from core snapshots switching from saving scripts to saving token streams in the VM isolate. R=asiva@google.com Committed: https://github.com/dart-lang/sdk/commit/22eee804cddfe768a575705caa613cc0d540cbc3

Patch Set 1 #

Patch Set 2 : format #

Total comments: 2

Patch Set 3 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -35 lines) Patch
M runtime/vm/clustered_snapshot.h View 1 chunk +1 line, -0 lines 0 comments Download
M runtime/vm/clustered_snapshot.cc View 1 2 5 chunks +15 lines, -2 lines 0 comments Download
M runtime/vm/snapshot.cc View 1 chunk +0 lines, -33 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
rmacnak
3 years, 8 months ago (2017-04-03 20:57:30 UTC) #2
siva
lgtm https://codereview.chromium.org/2790243003/diff/20001/runtime/vm/clustered_snapshot.cc File runtime/vm/clustered_snapshot.cc (right): https://codereview.chromium.org/2790243003/diff/20001/runtime/vm/clustered_snapshot.cc#newcode5368 runtime/vm/clustered_snapshot.cc:5368: // VM that has loaded this snapshots, much ...
3 years, 8 months ago (2017-04-06 19:19:40 UTC) #3
rmacnak
https://codereview.chromium.org/2790243003/diff/20001/runtime/vm/clustered_snapshot.cc File runtime/vm/clustered_snapshot.cc (right): https://codereview.chromium.org/2790243003/diff/20001/runtime/vm/clustered_snapshot.cc#newcode5368 runtime/vm/clustered_snapshot.cc:5368: // VM that has loaded this snapshots, much like ...
3 years, 8 months ago (2017-04-06 20:09:48 UTC) #4
rmacnak
3 years, 8 months ago (2017-04-06 20:22:31 UTC) #6
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
22eee804cddfe768a575705caa613cc0d540cbc3 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698