# HG changeset patch # User kvn # Date 1248638381 25200 # Node ID dd0a4e1e219be34f30b422eb1520ddcc1da13c27 # Parent a94af87c3357f7764b0bf46f2c9bb2004cf3b5c0 6851386: assert(b->find_node(def) < j,"uses must follow definitions") Summary: Add additional check for a tight loop. Reviewed-by: never diff -r a94af87c3357 -r dd0a4e1e219b src/share/vm/opto/block.cpp --- a/src/share/vm/opto/block.cpp Fri Jul 24 12:40:40 2009 -0700 +++ b/src/share/vm/opto/block.cpp Sun Jul 26 12:59:41 2009 -0700 @@ -910,7 +910,16 @@ !(b->head()->is_Loop() && n->is_Phi()) && // See (+++) comment in reg_split.cpp !(n->jvms() != NULL && n->jvms()->is_monitor_use(k)) ) { - assert( b->find_node(def) < j, "uses must follow definitions" ); + bool is_loop = false; + if (n->is_Phi()) { + for( uint l = 1; l < def->req(); l++ ) { + if (n == def->in(l)) { + is_loop = true; + break; // Some kind of loop + } + } + } + assert( is_loop || b->find_node(def) < j, "uses must follow definitions" ); } if( def->is_SafePointScalarObject() ) { assert(_bbs[def->_idx] == b, "SafePointScalarObject Node should be at the same block as its SafePoint node");