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

[dean.long at oracle.com]

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.

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
>>> **
>>
>