Mercurial > hg > truffle
comparison src/share/vm/gc_implementation/concurrentMarkSweep/compactibleFreeListSpace.hpp @ 4708:3c648b9ad052
7121373: Clean up CollectedHeap::is_in
Summary: Fixed G1CollectedHeap::is_in, added tests, cleaned up comments and made Space::is_in pure virtual.
Reviewed-by: brutisso, tonyp, jcoomes
author | stefank |
---|---|
date | Wed, 14 Dec 2011 12:15:26 +0100 |
parents | 5a5ed80bea5b |
children | 441e946dc1af |
comparison
equal
deleted
inserted
replaced
4132:6d7d0790074d | 4708:3c648b9ad052 |
---|---|
334 return MemRegion(bottom(), | 334 return MemRegion(bottom(), |
335 BlockOffsetArrayUseUnallocatedBlock ? | 335 BlockOffsetArrayUseUnallocatedBlock ? |
336 unallocated_block() : end()); | 336 unallocated_block() : end()); |
337 } | 337 } |
338 | 338 |
339 // This is needed because the default implementation uses block_start() | |
340 // which can;t be used at certain times (for example phase 3 of mark-sweep). | |
341 // A better fix is to change the assertions in phase 3 of mark-sweep to | |
342 // use is_in_reserved(), but that is deferred since the is_in() assertions | |
343 // are buried through several layers of callers and are used elsewhere | |
344 // as well. | |
345 bool is_in(const void* p) const { | 339 bool is_in(const void* p) const { |
346 return used_region().contains(p); | 340 return used_region().contains(p); |
347 } | 341 } |
348 | 342 |
349 virtual bool is_free_block(const HeapWord* p) const; | 343 virtual bool is_free_block(const HeapWord* p) const; |