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

Side by Side Diff: src/objects-debug.cc

Issue 1276533003: [runtime] Store constructor function index on primitive maps. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix arm failures Created 5 years, 4 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
« no previous file with comments | « src/objects.cc ('k') | src/objects-inl.h » ('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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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/disasm.h" 7 #include "src/disasm.h"
8 #include "src/disassembler.h" 8 #include "src/disassembler.h"
9 #include "src/jsregexp.h" 9 #include "src/jsregexp.h"
10 #include "src/macro-assembler.h" 10 #include "src/macro-assembler.h"
(...skipping 246 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 void JSObject::JSObjectVerify() { 257 void JSObject::JSObjectVerify() {
258 VerifyHeapPointer(properties()); 258 VerifyHeapPointer(properties());
259 VerifyHeapPointer(elements()); 259 VerifyHeapPointer(elements());
260 260
261 if (HasSloppyArgumentsElements()) { 261 if (HasSloppyArgumentsElements()) {
262 CHECK(this->elements()->IsFixedArray()); 262 CHECK(this->elements()->IsFixedArray());
263 CHECK_GE(this->elements()->length(), 2); 263 CHECK_GE(this->elements()->length(), 2);
264 } 264 }
265 265
266 if (HasFastProperties()) { 266 if (HasFastProperties()) {
267 int actual_unused_property_fields = map()->inobject_properties() + 267 int actual_unused_property_fields = map()->GetInObjectProperties() +
268 properties()->length() - 268 properties()->length() -
269 map()->NextFreePropertyIndex(); 269 map()->NextFreePropertyIndex();
270 if (map()->unused_property_fields() != actual_unused_property_fields) { 270 if (map()->unused_property_fields() != actual_unused_property_fields) {
271 // This could actually happen in the middle of StoreTransitionStub 271 // This could actually happen in the middle of StoreTransitionStub
272 // when the new extended backing store is already set into the object and 272 // when the new extended backing store is already set into the object and
273 // the allocation of the MutableHeapNumber triggers GC (in this case map 273 // the allocation of the MutableHeapNumber triggers GC (in this case map
274 // is not updated yet). 274 // is not updated yet).
275 CHECK_EQ(map()->unused_property_fields(), 275 CHECK_EQ(map()->unused_property_fields(),
276 actual_unused_property_fields - JSObject::kFieldsAdded); 276 actual_unused_property_fields - JSObject::kFieldsAdded);
277 } 277 }
(...skipping 1020 matching lines...) Expand 10 before | Expand all | Expand 10 after
1298 1298
1299 // Both are done at the same time. 1299 // Both are done at the same time.
1300 CHECK_EQ(new_it.done(), old_it.done()); 1300 CHECK_EQ(new_it.done(), old_it.done());
1301 } 1301 }
1302 1302
1303 1303
1304 #endif // DEBUG 1304 #endif // DEBUG
1305 1305
1306 } // namespace internal 1306 } // namespace internal
1307 } // namespace v8 1307 } // namespace v8
OLDNEW
« no previous file with comments | « src/objects.cc ('k') | src/objects-inl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698