RFR: 8144067: Pass obj directly to G1ParScanThreadState::update_rs
Thomas Schatzl
thomas.schatzl at oracle.com
Wed Nov 25 19:00:17 UTC 2015
Hi,
On Wed, 2015-11-25 at 18:53 +0100, Erik Helin wrote:
> Hi all,
>
> this small patch is an optimization of the call to
> G1ParScanThreadState::update_rs, which is used during the object copy.
> All callers of update_rs (G1ParScanThreadState::do_oop_evac and
> G1ParScanClosure::do_oop_nv) have aleady decoded the field 'p', so instead of
> calling load_decode_heap_oop(p) again in update_rs, we can just pass the
> object 'obj' as parameter (update_rs will be inlined anyway, so the parameter
> won't actually be passed).
>
> Thanks to Thomas for finding this optimization.
>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8144067
>
> Webrev:
> http://cr.openjdk.java.net/~ehelin/8144067/webrev.00/
>
> Please note that the patch is based upon the patch for 8144052.
>
> Testing:
> - GC test suite with full verification
looks good.
Thomas
More information about the hotspot-gc-dev
mailing list