RFR 8049105: Move array component mirror to instance of java/lang/Class (hotspot part 2)

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Aug 27 17:20:36 UTC 2014



On 8/26/14 6:56 AM, Coleen Phillimore wrote:
> Summary: This removes component mirrors from arrayKlass metadata and the
> C2 intrinsic for them.
>
> I'm sending this out to hotspot-dev so I can get someone from the
> compiler to review this (again).  Chris Thalinger, you already saw this
> once, it's the same thing, but I had to hold this change until it was
> okay'd by licensees.   Can you re-review?

Looks fine to me.

>
> Also GC team note that Klass::oops_do() is now non-virtual.  Will that
> help?

Yes, it helps. It reduce virtual table size for Klass and the method 
could be inlined.

>
> Tested with jck, nsk and jtreg tests.  Also, ran refworkload with no
> difference in performance.

Thanks,
Vladimir

>
> open webrev at http://cr.openjdk.java.net/~coleenp/8049105/
> bug link https://bugs.openjdk.java.net/browse/JDK-8049105
>
> Thanks,
> Coleen


More information about the hotspot-dev mailing list