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

[Christian Tornqvist]

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