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