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

Unified Diff: src/parser.cc

Issue 14850006: Use mutable heapnumbers to store doubles in fields. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 7 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
Index: src/parser.cc
diff --git a/src/parser.cc b/src/parser.cc
index 33b5fab3fc8c6b9cd198a99384fe4bf60111d738..07ce001d27189143f6652d3dd5d96b0492d2ce11 100644
--- a/src/parser.cc
+++ b/src/parser.cc
@@ -4014,6 +4014,12 @@ void Parser::BuildObjectLiteralConstantProperties(
// runtime. The enumeration order is maintained.
Handle<Object> key = property->key()->handle();
Handle<Object> value = GetBoilerplateValue(property->value());
+
+ // Ensure objects with doubles are always treated as nested objects.
+ // TODO(verwaest): Remove once we can store them inline.
+ if (FLAG_track_fields && value->IsNumber()) {
+ depth_acc = Max(2, depth_acc);
+ }
danno 2013/05/07 13:04:47 *COUGH* Any other better way to do this? *COUGH* B
Toon Verwaest 2013/05/07 15:08:52 Done.
is_simple_acc = is_simple_acc && !value->IsUndefined();
// Keep track of the number of elements in the object literal and

Powered by Google App Engine
This is Rietveld 408576698