RFR: 8035746: Add missing Klass::oop_is_instanceClassLoader() function

[Stefan Karlsson]

On 2014-02-25 15:15, Mikael Gerdin wrote:
> Hi Stefan,
>
> On Tuesday 25 February 2014 14.43.49 Stefan Karlsson wrote:
>> Hi all,
>>
>> Please, review this small patch to add Klass::oop_is_instanceClassLoader().
>>
>> Webrev:
>> http://cr.openjdk.java.net/~stefank/8035746/webrev.00
> Code change looks good.

Thanks.

>
> There are a couple of places where we check whether or not a Klass is a
> ClassLoader, something like:
> if (klass->is_subclass_of(SystemDictionary::ClassLoader_klass()))
>
> These places could now be changed to
> klass->oop_is_ClassLoader(), saving a possibly expensive subtype check.
>
> Should we file an RFE to change these or do you want to change them straight
> away?

I'd prefer a separate RFE.

thanks,
StefanK

>
> /Mikael
>
>> RFE:
>> https://bugs.openjdk.java.net/browse/JDK-8035746
>>
>> thanks,
>> StefanK