RFR (s) 8047736: Remove JVM_GetClassLoader as no longer used

[Coleen Phillimore]

Thanks, Christian!
Coleen

On 10/2/14, 1:28 PM, Christian Tornqvist wrote:
> Looks good.
>
> Thanks,
> Christian
>
> -----Original Message-----
> From: hotspot-runtime-dev
> [mailto:hotspot-runtime-dev-bounces at openjdk.java.net] On Behalf Of Coleen
> Phillimore
> Sent: Wednesday, October 1, 2014 2:03 PM
> To: Frederic Parain; hotspot-runtime-dev
> Subject: Re: RFR (s) 8047736: Remove JVM_GetClassLoader as no longer used
>
>
> Thanks for pointing out the missing changes.   They are here.
>
> http://cr.openjdk.java.net/~coleenp/8047736_2/
>
> Ran jck_core tests on linux.  JPRT will run the rest.
>
> Thanks,
> Coleen
>
> On 10/1/14, 9:45 AM, Frederic Parain wrote:
>> Coleen,
>>
>> JVM_GetClassLoader symbol should also be removed from mapfiles:
>>
>> File:  hotspot/make/aix/makefiles/mapfile-vers-debug
>> line 106 :                 JVM_GetClassLoader;
>>
>> File:  hotspot/make/aix/makefiles/mapfile-vers-product
>> line 106 :                 JVM_GetClassLoader;
>>
>> File:  hotspot/make/bsd/makefiles/mapfile-vers-darwin-debug
>> line 104 :                 _JVM_GetClassLoader
>>
>> File:  hotspot/make/bsd/makefiles/mapfile-vers-darwin-product
>> line 104 :                 _JVM_GetClassLoader
>>
>> File:  hotspot/make/bsd/makefiles/mapfile-vers-debug
>> line 106 :                 JVM_GetClassLoader;
>>
>> File:  hotspot/make/bsd/makefiles/mapfile-vers-product
>> line 106 :                 JVM_GetClassLoader;
>>
>> File:  hotspot/make/linux/makefiles/mapfile-vers-debug
>> line 106 :                 JVM_GetClassLoader;
>>
>> File:  hotspot/make/linux/makefiles/mapfile-vers-product
>> line 106 :                 JVM_GetClassLoader;
>>
>> File:  hotspot/make/solaris/makefiles/mapfile-vers
>> line 106 :                 JVM_GetClassLoader;
>>
>>
>> Regards,
>>
>> Fred
>>
>> On 09/26/2014 11:27 PM, Coleen Phillimore wrote:
>>> Summary: Remove this function from hotspot.  It's already removed
>>> from the JDK.
>>>
>>> Related to the bug I just fixed in the change I made months ago, the
>>> JVM interface to get the classLoader of a java/lang/Class is no
>>> longer needed.  This change has been approved by Oracle's licensees
>>> and has been approved in the internal compatibility committee request.
>>>
>>> Tested with nsk testbase and jtreg regression tests.
>>>
>>> open webrev at http://cr.openjdk.java.net/~coleenp/8047736/
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8047736
>>>
>>> Thanks,
>>> Coleen
>>>
>