Request for Reviews(M): 7092905: C2: Keep track of the number of dead nodes

Roland Westrelin roland.westrelin at
Tue Dec 4 08:42:04 PST 2012


I Know this is pushed already but I notice one thing that confuses me in this change that I haven't seen discussed before:

Compile::check_node_count() now checks live_nodes() against MaxNodeLimit but check_node_count() is used during parsing and at this point live_nodes() is not correct (it's C->unique(), right?). So depending on the compilation phase we either check the number of uniquely created nodes or the number of live nodes against MaxNodeLimit. To me that makes MaxNodeLimit mostly irrelevant.


More information about the hotspot-compiler-dev mailing list