RFR: 8035746: Add missing Klass::oop_is_instanceClassLoader()	function
    Mikael Gerdin 
    mikael.gerdin at oracle.com
       
    Tue Feb 25 06:15:32 PST 2014
    
    
  
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.
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?
/Mikael
> 
> RFE:
> https://bugs.openjdk.java.net/browse/JDK-8035746
> 
> thanks,
> StefanK
    
    
More information about the hotspot-dev
mailing list