[9] RFR(M): 8040213: C2 does not put all modified nodes on IGVN worklist
Tobias Hartmann
tobias.hartmann at oracle.com
Fri Jul 4 06:50:53 UTC 2014
Hi,
please review the following patch for 8040213 that finishes Vladimir's
work [1].
Bug: https://bugs.openjdk.java.net/browse/JDK-8040213
Webrev: http://cr.openjdk.java.net/~thartmann/8040213/webrev.00/
*
**Problem*
Not all nodes that are modified during C2 optimization are put on the
IGVN worklist.
*Solution*
Verification code is added that checks if (i) modified nodes are put on
the IGVN worklist and (ii) modified nodes are processed by
PhaseIterGVN::transform_old(..). All code that fails verification during
testing is fixed.
*
**Testing:*
- JPRT
- Full CTW
- Nashorn + Octane
- UTE (full testlist, BigApps, JRuby)
- VM (NSK) testbase
- Refworkload (reference_server)
- VMSQE.adhoc.JPRT.full batch
Thanks,
Tobias
[1] http://cr.openjdk.java.net/~kvn/8040213/webrev/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140704/8ffd9d33/attachment-0001.html>
More information about the hotspot-compiler-dev
mailing list