RFR(XS) JDK-8024646: Remove LRG_List container, replace it with GrowableArray

[Niclas Adlertz]

> It looks good. I would suggest to remove the line in coalesce.hpp instead of commenting.
Thanks. That should be removed indeed.

> 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.
Sure.

Kind Regards,
Niclas Adlertz

On 2013-09-11 21:10, Vladimir Kozlov wrote:
> 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