RFR(XS) JDK-8024646: Remove LRG_List container, replace it with	GrowableArray
    Vladimir Kozlov 
    vladimir.kozlov at oracle.com
       
    Wed Sep 11 12:10:57 PDT 2013
    
    
  
Niclas,
It looks good. I would suggest to remove the line in coalesce.hpp 
instead of commenting.
Could you file and fix a RFE about caching 
Thread::current()->resource_area() in Phase class?
There are a LOT of places in compiler where we use it and calling 
Thread::current() is not cheap.
Thanks,
Vladimir
On 9/11/13 11:09 AM, Niclas Adlertz wrote:
> Hi all,
>
> While working on 8022783, I've found that LRG_List can be removed.
> LRG_List is just another growable array, so we can replace it with the
> more common implementation named class GrowableArray.
>
> A review would be much appreciated.
>
> WEBREV: http://cr.openjdk.java.net/~adlertz/JDK-8024646/webrev00/
> JBS: https://bugs.openjdk.java.net/browse/JDK-8024646
>
> Kind Regards,
> Niclas Adlertz
    
    
More information about the hotspot-compiler-dev
mailing list