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


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.


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