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

Side by Side Diff: src/heap-snapshot-generator.cc

Issue 300283002: Introduce FieldIndex to unify and abstract property/field offset (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Fix mutable boxed double runtime function Created 6 years, 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/field-index-inl.h ('k') | src/ia32/stub-cache-ia32.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/v8.h" 5 #include "src/v8.h"
6 6
7 #include "src/heap-snapshot-generator-inl.h" 7 #include "src/heap-snapshot-generator-inl.h"
8 8
9 #include "src/allocation-tracker.h" 9 #include "src/allocation-tracker.h"
10 #include "src/code-stubs.h" 10 #include "src/code-stubs.h"
(...skipping 1642 matching lines...) Expand 10 before | Expand all | Expand 10 after
1653 NULL, 1653 NULL,
1654 js_obj->GetInObjectPropertyOffset(index)); 1654 js_obj->GetInObjectPropertyOffset(index));
1655 } else { 1655 } else {
1656 TagObject(value, "(hidden properties)"); 1656 TagObject(value, "(hidden properties)");
1657 SetInternalReference( 1657 SetInternalReference(
1658 js_obj, entry, 1658 js_obj, entry,
1659 "hidden_properties", value, 1659 "hidden_properties", value,
1660 js_obj->GetInObjectPropertyOffset(index)); 1660 js_obj->GetInObjectPropertyOffset(index));
1661 } 1661 }
1662 } else { 1662 } else {
1663 Object* value = js_obj->RawFastPropertyAt(index); 1663 FieldIndex field_index =
1664 FieldIndex::ForDescriptor(js_obj->map(), i);
1665 Object* value = js_obj->RawFastPropertyAt(field_index);
1664 if (k != heap_->hidden_string()) { 1666 if (k != heap_->hidden_string()) {
1665 SetPropertyReference(js_obj, entry, k, value); 1667 SetPropertyReference(js_obj, entry, k, value);
1666 } else { 1668 } else {
1667 TagObject(value, "(hidden properties)"); 1669 TagObject(value, "(hidden properties)");
1668 SetInternalReference(js_obj, entry, "hidden_properties", value); 1670 SetInternalReference(js_obj, entry, "hidden_properties", value);
1669 } 1671 }
1670 } 1672 }
1671 break; 1673 break;
1672 } 1674 }
1673 case CONSTANT: 1675 case CONSTANT:
(...skipping 1495 matching lines...) Expand 10 before | Expand all | Expand 10 after
3169 writer_->AddString("\"<dummy>\""); 3171 writer_->AddString("\"<dummy>\"");
3170 for (int i = 1; i < sorted_strings.length(); ++i) { 3172 for (int i = 1; i < sorted_strings.length(); ++i) {
3171 writer_->AddCharacter(','); 3173 writer_->AddCharacter(',');
3172 SerializeString(sorted_strings[i]); 3174 SerializeString(sorted_strings[i]);
3173 if (writer_->aborted()) return; 3175 if (writer_->aborted()) return;
3174 } 3176 }
3175 } 3177 }
3176 3178
3177 3179
3178 } } // namespace v8::internal 3180 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/field-index-inl.h ('k') | src/ia32/stub-cache-ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698