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

Side by Side Diff: test/cctest/heap/test-spaces.cc

Issue 2321553002: [heap] Do more incremental marking work in tasks. (Closed)
Patch Set: Created 4 years, 3 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 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 442
443 delete compaction_space; 443 delete compaction_space;
444 delete old_space; 444 delete old_space;
445 445
446 memory_allocator->TearDown(); 446 memory_allocator->TearDown();
447 delete memory_allocator; 447 delete memory_allocator;
448 } 448 }
449 449
450 450
451 TEST(LargeObjectSpace) { 451 TEST(LargeObjectSpace) {
452 // This test does not initialize allocated objects, which confuses the
453 // incremental marker.
454 FLAG_incremental_marking = false;
452 v8::V8::Initialize(); 455 v8::V8::Initialize();
453 456
454 LargeObjectSpace* lo = CcTest::heap()->lo_space(); 457 LargeObjectSpace* lo = CcTest::heap()->lo_space();
455 CHECK(lo != NULL); 458 CHECK(lo != NULL);
456 459
457 int lo_size = Page::kPageSize; 460 int lo_size = Page::kPageSize;
458 461
459 Object* obj = lo->AllocateRaw(lo_size, NOT_EXECUTABLE).ToObjectChecked(); 462 Object* obj = lo->AllocateRaw(lo_size, NOT_EXECUTABLE).ToObjectChecked();
460 CHECK(obj->IsHeapObject()); 463 CHECK(obj->IsHeapObject());
461 464
(...skipping 325 matching lines...) Expand 10 before | Expand all | Expand 10 after
787 HeapObject* filler = 790 HeapObject* filler =
788 HeapObject::FromAddress(array->address() + array->Size()); 791 HeapObject::FromAddress(array->address() + array->Size());
789 CHECK_EQ(filler->map(), CcTest::heap()->two_pointer_filler_map()); 792 CHECK_EQ(filler->map(), CcTest::heap()->two_pointer_filler_map());
790 793
791 const size_t shrinked = page->ShrinkToHighWaterMark(); 794 const size_t shrinked = page->ShrinkToHighWaterMark();
792 CHECK_EQ(0, shrinked); 795 CHECK_EQ(0, shrinked);
793 } 796 }
794 797
795 } // namespace internal 798 } // namespace internal
796 } // namespace v8 799 } // namespace v8
OLDNEW
« src/heap/heap.cc ('K') | « test/cctest/heap/test-incremental-marking.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698