RFR(S) 8009721: Move the definition of class LRG_List

Niclas Adlertz niclas.adlertz at oracle.com
Wed Mar 13 07:10:46 PDT 2013


Thank you Roland, Rickard, Nils and Vladimir.

Regards,
Niclas Adlertz

On 12 mar 2013, at 19:39, Vladimir Kozlov wrote:

> Good.
> 
> Vladimir
> 
> On 3/12/13 5:51 AM, Niclas Adlertz wrote:
>> Hi all.
>> 
>> Problem:
>> The class PhaseLive in opto/live.hpp uses a data structure, class LRG_List, defined in opto/chaitin.hpp. Each time the class PhaseLive is used, chaitin.hpp needs to be included. This would make no sense if, for example, a new register allocator in C2 would like to use the class PhaseLive and because of this needs to include chaitin.hpp.
>> 
>> Solution:
>> Move the definition of the class LRG_List from opto/chaitin.hpp to opto/live.hpp.
>> 
>> A review would be much appreciated.
>> Thank you.
>> 
>> Regards,
>> Niclas Adlertz
>> 
>> ---------------------
>> 
>> WEBREV: http://cr.openjdk.java.net/~neliasso/8009721/webrev.01/
>> BUG: https://jbs.oracle.com/bugs/browse/JDK-8009721, http://bugs.sun.com/view_bug.do?bug_id=8009721
>> 



More information about the hotspot-compiler-dev mailing list