RFR (S) 8150669: C1 intrinsic for Class.isPrimitive

Christian Thalinger christian.thalinger at oracle.com
Tue Mar 1 00:37:26 UTC 2016


> On Feb 29, 2016, at 12:59 PM, Aleksey Shipilev <aleksey.shipilev at oracle.com> wrote:
> 
> On 03/01/2016 01:35 AM, Christian Thalinger wrote:
>>> On Feb 29, 2016, at 11:58 AM, Aleksey Shipilev
>>> <aleksey.shipilev at oracle.com> wrote: See the notes at the bottom: 
>>> http://cr.openjdk.java.net/~shade/8150669/notes.txt
>> 
>> That’s good.  I wonder why this wasn’t intrinsified before.
> 
> Yup, puzzled me too.
> 
>> One nit:  can we rename IsPrimitive.java to
>> TestClassIsPrimitive.java?  Most of our tests are called Test* while
>> support files have other names.  Not sure if we have a
>> hard-convention on this but it certainly is nice and helpful
>> sometimes.
> 
> Yes, we can:
> http://cr.openjdk.java.net/~shade/8150669/webrev.02/

Thanks.  Looks good.

> 
> -Aleksey
> 



More information about the hotspot-compiler-dev mailing list