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

[Frederic Parain]

Coleen,

Looks good to me now,

thank you,

Fred

On 01/10/2014 20:03, Coleen Phillimore wrote:
>
> 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
>>>
>>
>

-- 
Frederic Parain - Oracle
Grenoble Engineering Center - France
Phone: +33 4 76 18 81 17
Email: Frederic.Parain at oracle.com