Code Review for hotspot

Prepared by:roland on Mon Nov 18 21:54:11 CET 2013
Workspace:/net/mackdrive/export/jrts/work/rw140007/c2-speculation/hotspot
Compare against: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot
Compare against version:deoptfix
Summary of changes: 198 lines changed: 108 ins; 3 del; 87 mod; 35971 unchg
Patch of changes: hotspot.patch
Legend: Modified file
Deleted file
New file

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 1193 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
4 lines changed: 0 ins; 0 del; 4 mod; 2229 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
25 lines changed: 16 ins; 1 del; 8 mod; 3946 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
4 lines changed: 0 ins; 0 del; 4 mod; 1374 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
4 lines changed: 0 ins; 0 del; 4 mod; 4115 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
4 lines changed: 0 ins; 0 del; 4 mod; 2789 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 4582 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 208 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 2131 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 2191 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
5 lines changed: 0 ins; 0 del; 5 mod; 2398 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 537 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
3 lines changed: 0 ins; 1 del; 2 mod; 1805 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
53 lines changed: 20 ins; 1 del; 32 mod; 4801 unchg

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

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
23 lines changed: 9 ins; 0 del; 14 mod; 1672 unchg

------ ------ ------ ------ --- New Patch Raw test/compiler/types/TestSpeculationFailedHigherEqual.java

rev 5645 : 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
Summary: type methods shouldn't always operate on speculative part
Reviewed-by:
63 lines changed: 63 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using ../../c2-inlining/webrev.ksh (vers 23.18-hg-never-dev).