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

Rickard Bäckman rickard.backman at oracle.com
Tue Mar 12 06:11:14 PDT 2013


Looks good to me.

/R

On Mar 12, 2013, at 1:51 PM, 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