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

[Coleen Phillimore]

Thank you, Vladimir!

Coleen

On 8/27/14, 1:20 PM, Vladimir Kozlov wrote:
>
>
> 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