RFR (S): 8035326: Assume non-NULL references in G1CollectedHeap::in_cset_fast_test

Jon Masamitsu jon.masamitsu at oracle.com
Wed Feb 19 17:18:07 PST 2014


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), "");


Jon

On 2/19/2014 7:12 AM, Thomas Schatzl wrote:
> Hi all,
>
>    can I get reviews for the following change that lifts the assumption
> that the oop* passed to G1CollectedHeap::in_cset_fast_test is non-null,
> allowing manual optimization of code.
>
> Almost all uses of G1CollectedHeap::in_cset_fast_test() already made
> sure that non-null references are passed to it. The only remaining one,
> in G1ParCopyClosure::do_oop_work() actually benefits from lifting this
> restriction by allowing removal of some additional checks.
>
> Also removes another superfluous check in the same method.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8035326
>
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8035326/webrev/
>
> This is based on the change for 8027559.
>
> Testing:
> jprt
>
>
>
> Thanks,
>    Thomas
>



More information about the hotspot-gc-dev mailing list