Code Review for hotspot

Prepared by:neliasso on Fri Sep 25 16:23:27 CEST 2015
Workspace:/home/neliasso/repos/compiler_control/hotspot
Compare against: ssh://hg.openjdk.java.net/jdk9/hs-comp/hotspot
Compare against version:9002
Summary of changes: 3633 lines changed: 3382 ins; 163 del; 88 mod; 66114 unchg
Patch of changes: hotspot.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/c1/c1_Compilation.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
8 lines changed: 3 ins; 2 del; 3 mod; 703 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/c1/c1_Compilation.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
3 lines changed: 2 ins; 0 del; 1 mod; 344 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/c1/c1_Compiler.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 255 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/c1/c1_Compiler.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
2 lines changed: 1 ins; 0 del; 1 mod; 64 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/c1/c1_GraphBuilder.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
8 lines changed: 2 ins; 1 del; 5 mod; 4263 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/ci/ciEnv.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
17 lines changed: 10 ins; 3 del; 4 mod; 1277 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/ci/ciEnv.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
14 lines changed: 9 ins; 5 del; 0 mod; 464 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/ci/ciMethod.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
55 lines changed: 0 ins; 54 del; 1 mod; 1414 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/ci/ciMethod.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
11 lines changed: 2 ins; 8 del; 1 mod; 337 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/classfile/vmSymbols.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
9 lines changed: 0 ins; 8 del; 1 mod; 911 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/classfile/vmSymbols.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 1402 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/code/nmethod.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
15 lines changed: 0 ins; 14 del; 1 mod; 3126 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/compiler/abstractCompiler.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
5 lines changed: 2 ins; 0 del; 3 mod; 140 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/compiler/compileBroker.cpp

165 lines changed: 138 ins; 23 del; 4 mod; 2050 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/compiler/compileBroker.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
27 lines changed: 24 ins; 1 del; 2 mod; 364 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/compiler/compileTask.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
14 lines changed: 14 ins; 0 del; 0 mod; 391 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/compiler/compilerOracle.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
19 lines changed: 8 ins; 7 del; 4 mod; 881 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/compiler/compilerOracle.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
5 lines changed: 4 ins; 0 del; 1 mod; 87 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/compiler/methodMatcher.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
95 lines changed: 95 ins; 0 del; 0 mod; 347 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/compiler/methodMatcher.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
57 lines changed: 28 ins; 27 del; 2 mod; 97 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
2 lines changed: 1 ins; 0 del; 1 mod; 1769 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
2 lines changed: 2 ins; 0 del; 0 mod; 891 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
3 lines changed: 0 ins; 0 del; 3 mod; 732 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
2 lines changed: 0 ins; 0 del; 2 mod; 707 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
3 lines changed: 1 ins; 0 del; 2 mod; 434 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
2 lines changed: 1 ins; 0 del; 1 mod; 70 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 2408 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
24 lines changed: 5 ins; 2 del; 17 mod; 4397 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
11 lines changed: 2 ins; 3 del; 6 mod; 1288 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
5 lines changed: 0 ins; 2 del; 3 mod; 728 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 140 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
4 lines changed: 2 ins; 1 del; 1 mod; 6340 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
3 lines changed: 1 ins; 1 del; 1 mod; 2887 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 2390 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
6 lines changed: 4 ins; 0 del; 2 mod; 1622 unchg

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

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 3906 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/jni.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
4 lines changed: 4 ins; 0 del; 0 mod; 4371 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/prims/whitebox.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
41 lines changed: 39 ins; 0 del; 2 mod; 1550 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/globals.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
10 lines changed: 9 ins; 0 del; 1 mod; 4199 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/init.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
4 lines changed: 2 ins; 0 del; 2 mod; 187 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/mutexLocker.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
2 lines changed: 2 ins; 0 del; 0 mod; 327 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/mutexLocker.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
1 line changed: 1 ins; 0 del; 0 mod; 369 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/sharedRuntime.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
6 lines changed: 6 ins; 0 del; 0 mod; 2848 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/runtime/vm_operations.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
1 line changed: 0 ins; 0 del; 1 mod; 497 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/services/diagnosticCommand.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
42 lines changed: 42 ins; 0 del; 0 mod; 900 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/services/diagnosticCommand.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
86 lines changed: 86 ins; 0 del; 0 mod; 616 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/shark/sharkCompiler.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
4 lines changed: 2 ins; 1 del; 1 mod; 368 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/shark/sharkCompiler.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
2 lines changed: 1 ins; 0 del; 1 mod; 133 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/serviceability/dcmd/compiler/CompilerQueueTest.java

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
1 line changed: 1 ins; 0 del; 0 mod; 123 unchg

------ ------ ------ ------ --- New Patch Raw src/share/vm/compiler/compilerDirectives.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
351 lines changed: 351 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw src/share/vm/compiler/compilerDirectives.hpp

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

------ ------ ------ ------ --- New Patch Raw src/share/vm/compiler/directivesParser.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
738 lines changed: 738 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw src/share/vm/compiler/directivesParser.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
138 lines changed: 138 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw src/share/vm/utilities/json.cpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
955 lines changed: 955 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw src/share/vm/utilities/json.hpp

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
125 lines changed: 125 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/compiler/compilercontrol/InlineMatcherTest.java

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
167 lines changed: 167 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/serviceability/dcmd/compiler/CompilerDirectivesDCMDTest.java

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
154 lines changed: 154 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/serviceability/dcmd/compiler/control1.txt

rev 9003 : 8137167: JEP165: Compiler Control: Implementation task
Summary:
Reviewed-by:
23 lines changed: 23 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /home/neliasso/tools/webrev.ksh (vers 23.18-hg-never-dev).