Code Review for hs-gc9

The HeapReagionSet hierarchy has been simplified. Instead of subclasses aggregation is used for the MT-safety checks.

BeforeAfter
Prepared by:brutisso on Tue Feb 11 11:36:04 CET 2014
Workspace:/localhome/repos/hs-gc9
Compare against: http://hg.openjdk.java.net/jdk9/hs-gc/hotspot
Compare against version:5874
Summary of changes: 1087 lines changed: 227 ins; 645 del; 215 mod; 16916 unchg
Patch of changes: hs-gc9.patch
Bug id: JDK-8034079 G1: Refactor the HeapRegionSet hierarchy
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw agent/src/share/classes/sun/jvm/hotspot/gc_implementation/g1/G1CollectedHeap.java

2 lines changed: 0 ins; 0 del; 2 mod; 132 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw agent/src/share/classes/sun/jvm/hotspot/gc_implementation/g1/HeapRegionSetBase.java

21 lines changed: 2 ins; 13 del; 6 mod; 62 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw agent/src/share/classes/sun/jvm/hotspot/tools/HeapSummary.java

2 lines changed: 1 ins; 0 del; 1 mod; 300 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/concurrentMark.cpp

39 lines changed: 11 ins; 10 del; 18 mod; 4665 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/concurrentMark.hpp

1 line changed: 0 ins; 0 del; 1 mod; 1287 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp

132 lines changed: 31 ins; 57 del; 44 mod; 6841 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp

74 lines changed: 20 ins; 45 del; 9 mod; 2044 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/g1CollectedHeap.inline.hpp

1 line changed: 1 ins; 0 del; 0 mod; 226 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/g1MarkSweep.cpp

14 lines changed: 3 ins; 4 del; 7 mod; 354 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/heapRegionSeq.cpp

1 line changed: 0 ins; 0 del; 1 mod; 260 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/heapRegionSet.cpp

224 lines changed: 4 ins; 175 del; 45 mod; 250 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/heapRegionSet.hpp

224 lines changed: 60 ins; 132 del; 32 mod; 219 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/heapRegionSet.inline.hpp

76 lines changed: 1 ins; 55 del; 20 mod; 92 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/heapRegionSets.cpp

87 lines changed: 18 ins; 43 del; 26 mod; 106 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/gc_implementation/g1/vmStructs_g1.hpp

5 lines changed: 2 ins; 0 del; 3 mod; 78 unchg

------ ------ ------ ------ --- New Patch Raw agent/src/share/classes/sun/jvm/hotspot/gc_implementation/g1/HeapRegionSetCount.java

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

------ ------ ------ ------ Old --- Patch --- src/share/vm/gc_implementation/g1/heapRegionSets.hpp

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

This code review page was prepared using /localhome/tools/tom-webrev (vers 23.18-hg-never-dev).