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

Side by Side Diff: src/incremental-marking.cc

Issue 11085070: Enable --verify-heap in release mode (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: After rebase plus one new issue fix Created 8 years, 2 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
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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 543 matching lines...) Expand 10 before | Expand all | Expand 10 after
554 EnsureMarkingDequeIsCommitted(); 554 EnsureMarkingDequeIsCommitted();
555 555
556 // Initialize marking stack. 556 // Initialize marking stack.
557 Address addr = static_cast<Address>(marking_deque_memory_->address()); 557 Address addr = static_cast<Address>(marking_deque_memory_->address());
558 size_t size = marking_deque_memory_->size(); 558 size_t size = marking_deque_memory_->size();
559 if (FLAG_force_marking_deque_overflows) size = 64 * kPointerSize; 559 if (FLAG_force_marking_deque_overflows) size = 64 * kPointerSize;
560 marking_deque_.Initialize(addr, addr + size); 560 marking_deque_.Initialize(addr, addr + size);
561 561
562 ActivateIncrementalWriteBarrier(); 562 ActivateIncrementalWriteBarrier();
563 563
564 #ifdef DEBUG
565 // Marking bits are cleared by the sweeper. 564 // Marking bits are cleared by the sweeper.
566 if (FLAG_verify_heap) { 565 if (FLAG_verify_heap) {
567 heap_->mark_compact_collector()->VerifyMarkbitsAreClean(); 566 heap_->mark_compact_collector()->VerifyMarkbitsAreClean();
568 } 567 }
569 #endif
570 568
571 heap_->CompletelyClearInstanceofCache(); 569 heap_->CompletelyClearInstanceofCache();
572 heap_->isolate()->compilation_cache()->MarkCompactPrologue(); 570 heap_->isolate()->compilation_cache()->MarkCompactPrologue();
573 571
574 if (FLAG_cleanup_code_caches_at_gc) { 572 if (FLAG_cleanup_code_caches_at_gc) {
575 // We will mark cache black with a separate pass 573 // We will mark cache black with a separate pass
576 // when we finish marking. 574 // when we finish marking.
577 MarkObjectGreyDoNotEnqueue(heap_->polymorphic_code_cache()); 575 MarkObjectGreyDoNotEnqueue(heap_->polymorphic_code_cache());
578 } 576 }
579 577
(...skipping 371 matching lines...) Expand 10 before | Expand all | Expand 10 after
951 bytes_scanned_ = 0; 949 bytes_scanned_ = 0;
952 write_barriers_invoked_since_last_step_ = 0; 950 write_barriers_invoked_since_last_step_ = 0;
953 } 951 }
954 952
955 953
956 int64_t IncrementalMarking::SpaceLeftInOldSpace() { 954 int64_t IncrementalMarking::SpaceLeftInOldSpace() {
957 return heap_->MaxOldGenerationSize() - heap_->PromotedSpaceSizeOfObjects(); 955 return heap_->MaxOldGenerationSize() - heap_->PromotedSpaceSizeOfObjects();
958 } 956 }
959 957
960 } } // namespace v8::internal 958 } } // namespace v8::internal
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698