RFR (S): 8035326: Assume non-NULL references in G1CollectedHeap::in_cset_fast_test
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Feb 20 10:17:39 UTC 2014
Hi Jon,
thanks for the review.
On Wed, 2014-02-19 at 17:18 -0800, Jon Masamitsu wrote:
> Thomas,
>
> Changes look good.
>
> Is there really nothing you would put into the assert string?
>
> http://cr.openjdk.java.net/~tschatzl/8035326/webrev/src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp.udiff.html
>
> + assert(_g1_committed.contains((HeapWord*) obj), "");
>
Fixed in http://cr.openjdk.java.net/~tschatzl/8035326/webrev.1 along
with other suggestions from Stefan.
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list