Code Review for 6663848

Prepared by: Thomas Rodriguez (never) on Wed Apr 23 14:13:02 PDT 2008
Workspace:/export/ws/hs13
Compare against: ssh://hg.openjdk.java.net/jdk7/hotspot-comp-gate/hotspot
Compare against version:113
Summary of changes: 529 lines changed: 512 ins; 7 del; 10 mod; 4927 unchg
Patch of changes: 6663848.patch
Author comments:
6663848: assert(i < Max(),"oob") in C2 with -Xcomp
Summary:
Reviewed-by:

NeverBranchNodes are used to hook infinite loops up to the root so
that the code doesn't appear to be dead. Currently they don't contain
any code to allow them to fold up when they truly become dead. This
can result in a malformed one being used during code generation
resulting in assertion failures. Additionally it should participate
in the expected_kids check final_graph_reshaping and I cleaned up the
logic for this as well.

Bug id: 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/cfgnode.hpp

7 lines changed: 6 ins; 0 del; 1 mod; 512 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/cfgnode.cpp

22 lines changed: 22 ins; 0 del; 0 mod; 2024 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/opto/compile.cpp

20 lines changed: 4 ins; 7 del; 9 mod; 2391 unchg

------ ------ ------ ------ --- New Patch Raw test/compiler/6663848/Tester.java

480 lines changed: 480 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /net/smite.sfbay/never/bin/hgwebrev (vers 23.12-hg-never).