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

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

Issue 1705713002: [runtime] Replace hidden_string with a 0-hash-code private symbol (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 10 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
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/objects.h" 5 #include "src/objects.h"
6 6
7 #include "src/bootstrapper.h" 7 #include "src/bootstrapper.h"
8 #include "src/disasm.h" 8 #include "src/disasm.h"
9 #include "src/disassembler.h" 9 #include "src/disassembler.h"
10 #include "src/field-type.h" 10 #include "src/field-type.h"
(...skipping 190 matching lines...) Expand 10 before | Expand all | Expand 10 after
201 void HeapObject::VerifyHeapPointer(Object* p) { 201 void HeapObject::VerifyHeapPointer(Object* p) {
202 CHECK(p->IsHeapObject()); 202 CHECK(p->IsHeapObject());
203 HeapObject* ho = HeapObject::cast(p); 203 HeapObject* ho = HeapObject::cast(p);
204 CHECK(ho->GetHeap()->Contains(ho)); 204 CHECK(ho->GetHeap()->Contains(ho));
205 } 205 }
206 206
207 207
208 void Symbol::SymbolVerify() { 208 void Symbol::SymbolVerify() {
209 CHECK(IsSymbol()); 209 CHECK(IsSymbol());
210 CHECK(HasHashCode()); 210 CHECK(HasHashCode());
211 CHECK_GT(Hash(), 0u); 211 CHECK(GetHeap()->hidden_properties_symbol() == this || Hash() > 0u);
212 CHECK(name()->IsUndefined() || name()->IsString()); 212 CHECK(name()->IsUndefined() || name()->IsString());
213 } 213 }
214 214
215 215
216 void HeapNumber::HeapNumberVerify() { 216 void HeapNumber::HeapNumberVerify() {
217 CHECK(IsHeapNumber() || IsMutableHeapNumber()); 217 CHECK(IsHeapNumber() || IsMutableHeapNumber());
218 } 218 }
219 219
220 220
221 void Simd128Value::Simd128ValueVerify() { CHECK(IsSimd128Value()); } 221 void Simd128Value::Simd128ValueVerify() { CHECK(IsSimd128Value()); }
(...skipping 1098 matching lines...) Expand 10 before | Expand all | Expand 10 after
1320 1320
1321 // Both are done at the same time. 1321 // Both are done at the same time.
1322 CHECK_EQ(new_it.done(), old_it.done()); 1322 CHECK_EQ(new_it.done(), old_it.done());
1323 } 1323 }
1324 1324
1325 1325
1326 #endif // DEBUG 1326 #endif // DEBUG
1327 1327
1328 } // namespace internal 1328 } // namespace internal
1329 } // namespace v8 1329 } // namespace v8
OLDNEW
« src/heap/heap.cc ('K') | « src/objects.cc ('k') | src/profiler/heap-snapshot-generator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698