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

Coleen Phillimore coleen.phillimore at oracle.com
Thu Oct 2 16:15:29 UTC 2014


Thank you Fred!
Coleen

On 10/2/14, 8:30 AM, Frederic Parain wrote:
> 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
>>>>
>>>
>>
>



More information about the hotspot-runtime-dev mailing list