Integrated: 8254015: copy_to_survivor_space should use in-hand klass for scanning

[Kim Barrett]

On Mon, 5 Oct 2020 12:46:42 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this small change to G1 evacuation.  When scanning the oops
> in an instance, the existing code calls obj->oop_iterate_backward(closure).
> That function gets the klass of obj and forwards to another function with
> the obj, closure, and the just obtained klass.  But the caller of
> oop_iterate_backward has already obtained the klass for other reasons.
> Rather than refetching it (including dealing with
> UseCompressedClassPointers), change to use a new oop_iterate_backward
> overload taking the klass.  This saves a small amount of work for non-array
> objects being evacuated.
> 
> Testing:
> tier1
> Generic performance testing didn't find any significant difference.
> Looking at just GCBasher evacuation rate, there may to be a small
> improvement, though it's hard to measure.

This pull request has now been integrated.

Changeset: 66f27b5f
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/66f27b5f
Stats:     11 lines in 3 files changed: 9 ins; 0 del; 2 mod

8254015: copy_to_survivor_space should use in-hand klass for scanning

Reviewed-by: tschatzl, sjohanss, ayang

-------------

PR: https://git.openjdk.java.net/jdk/pull/504