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

Unified Diff: src/objects.cc

Issue 2740033002: [serializer] complete slack tracking when creating context snapshot. (Closed)
Patch Set: remove unused variable. Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/api.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index 77c3ab82a7ad443c28e2383032a94bab6af18c2d..dc9bf1deb47f46998ffe2ff6a8df5e22bc8046e1 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -13577,16 +13577,9 @@ void SetExpectedNofPropertiesFromEstimate(Handle<SharedFunctionInfo> shared,
// to be added later.
if (estimate == 0) estimate = 2;
- // TODO(yangguo): check whether those heuristics are still up-to-date.
- // We do not shrink objects that go into a snapshot (yet), so we adjust
- // the estimate conservatively.
- if (shared->GetIsolate()->serializer_enabled()) {
- estimate += 2;
- } else {
- // Inobject slack tracking will reclaim redundant inobject space later,
- // so we can afford to adjust the estimate generously.
- estimate += 8;
- }
+ // Inobject slack tracking will reclaim redundant inobject space later,
+ // so we can afford to adjust the estimate generously.
+ estimate += 8;
shared->set_expected_nof_properties(estimate);
}
@@ -13639,13 +13632,7 @@ void SharedFunctionInfo::SetConstructStub(Code* code) {
void Map::StartInobjectSlackTracking() {
DCHECK(!IsInobjectSlackTrackingInProgress());
-
- // No tracking during the snapshot construction phase.
- Isolate* isolate = GetIsolate();
- if (isolate->serializer_enabled()) return;
-
if (unused_property_fields() == 0) return;
-
set_construction_counter(Map::kSlackTrackingCounterStart);
}
« no previous file with comments | « src/api.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698