Revision: 9347 Author: [email protected] Date: Tue Sep 20 07:46:33 2011 Log: Fixed an assertion in PagedSpace::Verify.
The formerly unused variable was found by GCC 4.6.1, BTW... Review URL: http://codereview.chromium.org/7979005 http://code.google.com/p/v8/source/detail?r=9347 Modified: /branches/bleeding_edge/src/spaces.cc ======================================= --- /branches/bleeding_edge/src/spaces.cc Tue Sep 20 06:37:40 2011 +++ /branches/bleeding_edge/src/spaces.cc Tue Sep 20 07:46:33 2011 @@ -785,7 +785,7 @@ if (was_swept_conservatively_) return; bool allocation_pointer_found_in_space = - (allocation_info_.top != allocation_info_.limit); + (allocation_info_.top == allocation_info_.limit); PageIterator page_iterator(this); while (page_iterator.has_next()) { Page* page = page_iterator.next(); @@ -826,6 +826,7 @@ CHECK_LE(black_size, page->LiveBytes()); } + ASSERT(allocation_pointer_found_in_space); } #endif -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
