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