RFR: 8216157: Enable inlining of java_lang_Class::is_primitive

Claes Redestad claes.redestad at oracle.com
Fri Jan 4 12:51:20 UTC 2019


Erik, Coleen, thanks for reviewing!

/Claes

On 2019-01-04 13:44, coleen.phillimore at oracle.com wrote:
> +1
> Coleen
> 
> On 1/4/19 5:53 AM, Erik Österlund wrote:
>> +1
>>
>> /Erik
>>
>> On 2019-01-04 11:17, Claes Redestad wrote:
>>> Hi,
>>>
>>> moving Class::is_primitive to javaClasses.inline.hpp turns out to be a
>>> micro-optimization on a number of operations, such as Array.newInstance
>>> (until the C2 intrinsic kicks in).
>>>
>>> As the method is quite trivial inlining seem to happen everywhere and
>>> still end up shrinking the VM size by a few bytes (linux-x64).
>>>
>>> Bug:    https://bugs.openjdk.java.net/browse/JDK-8216157
>>> Webrev: http://cr.openjdk.java.net/~redestad/8216157/open.00/
>>>
>>> Testing: tier1-3 in-flight
>>>
>>> /Claes
>>
> 


More information about the hotspot-runtime-dev mailing list