RFR: 8202479: Add missing try_resolve_jobject_in_native calls

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Mon May 7 13:51:50 UTC 2018


http://cr.openjdk.java.net/~eosterlund/8202479/webrev.00/src/hotspot/cpu/sparc/jniFastGetField_sparc.cpp.udiff.html

Generally G3 is seen as G3_scratch in sparc code.

Why don't both paths have G3 as tmp?
thanks,
Coleen

On 5/7/18 4:53 AM, Erik Österlund wrote:
> Ping!
>
> On 2018-05-01 17:00, Erik Österlund wrote:
>> Hi,
>>
>> There are some missing calls to try_resolve_jobject_in_native for the 
>> jni fast get field optimization.
>>
>> On x86_64, it is used for T_BOOLEAN, T_BYTE, T_CHAR, T_SHORT, T_INT 
>> and T_LONG, but is missing for T_FLOAT and T_DOUBLE.
>> On SPARC, it is used for T_BOOLEAN, T_BYTE, T_CHAR, T_SHORT, T_INT, 
>> but is missing for T_LONG, T_FLOAT and T_DOUBLE.
>> On AArch64, it is used for all types.
>>
>> Here is a patch to add the missing calls to 
>> try_resolve_jobject_in_native.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~eosterlund/8202479/webrev.00/
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8202479
>>
>> Thanks,
>> /Erik
>



More information about the hotspot-dev mailing list