Request for reviews (S): 7050280: assert(u->as_Unlock()->is_eliminated()) failed: sanity

Vladimir Kozlov vladimir.kozlov at oracle.com
Thu Jun 2 19:25:25 PDT 2011


http://cr.openjdk.java.net/~kvn/7050280/webrev

Fixed 7050280: assert(u->as_Unlock()->is_eliminated()) failed: sanity

Marking locks for elimination is done in IGVN. Unfortunately the order of IGVN 
worklist processing may affect this marking. Also during EA obj may point to 
several objects but after few ideal graph transformations (CCP) it may point to 
only one non escaping object (but still using phi), corresponding locks and 
unlocks will be marked for elimination. Later obj could be replaced with a new 
node (new phi) and will be no escape information about it. And later after some 
graph reshape other locks and unlocks (which were not marked for elimination 
before) are connected to this new obj but they still will not be marked for 
elimination since new obj has no escape information.

Fix:
Do first round of locks marking during EA after ConnectionGraph is constructed 
when all nodes have escape information. During Macro nodes expansion move 
creation of new eliminated BoxLock into new separate method which is called 
before locks elimination. Use this method to mark all associated (same box and 
obj) lock and unlock nodes which were not marked before (instead of the assert).

Tested with failing case and full CTW.


More information about the hotspot-compiler-dev mailing list