RFR (S) 8214972: Uses of klass_holder() except GC need to apply GC barriers

[coleen.phillimore at oracle.com]

On 12/7/18 2:58 PM, dean.long at oracle.com wrote:
> On 12/7/18 5:09 AM, coleen.phillimore at oracle.com wrote:
>>
>>
>> On 12/6/18 10:29 PM, dean.long at oracle.com wrote:
>>> It looks like we no longer use java_mirror() for 
>>> is_unsafe_anonymous() classes?
>>
>> Hi Dean,  I'm glad you looked at this.  The ClassLoaderData::_holder 
>> is the java mirror now, so we only have to look at that, and mark it 
>> for the concurrent collectors.
>>
>
> OK, I missed the earlier ClassLoaderData::_holder change.  This change 
> looks good.

Thank you very much Dean!
Coleen
>
> dl
>
>> Coleen
>>
>>>
>>> dl
>>>
>>>
>>> On 12/6/18 12:47 PM, coleen.phillimore at oracle.com wrote:
>>>> Summary: Fix klass_holder() and make all callers use it, remove 
>>>> holder_phantom().*
>>>>
>>>> *This removes the confusion about which one to use, except for GC, 
>>>> who has to know.
>>>>
>>>> open webrev at http://cr.openjdk.java.net/~coleenp/8214972.01/webrev
>>>> bug link https://bugs.openjdk.java.net/browse/JDK-8214972
>>>>
>>>> Ran tier1-3 tests.
>>>>
>>>> Thanks,
>>>> Coleen
>>>> **
>>>
>>
>