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