Code Review for hotspot

Prepared by:neliasso on Tue Sep 22 16:26:57 CEST 2015
Workspace:/home/neliasso/repos/compiler_control/hotspot
Compare against: ssh://hg.openjdk.java.net/jdk9/hs-comp/hotspot
Compare against version:8994
Summary of changes: 3429 lines changed: 3201 ins; 156 del; 72 mod; 63809 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 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
2 lines changed: 1 ins; 0 del; 1 mod; 707 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
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/c1/c1_LIRGenerator.cpp

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
3 lines changed: 1 ins; 2 del; 0 mod; 3669 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
27 lines changed: 23 ins; 2 del; 2 mod; 1280 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
15 lines changed: 10 ins; 4 del; 1 mod; 464 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
17 lines changed: 3 ins; 11 del; 3 mod; 3127 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
3 lines changed: 1 ins; 0 del; 2 mod; 141 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
176 lines changed: 148 ins; 23 del; 5 mod; 2049 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
28 lines changed: 25 ins; 1 del; 2 mod; 364 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
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/compileTask.hpp

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
2 lines changed: 2 ins; 0 del; 0 mod; 151 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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/chaitin.cpp

rev 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
17 lines changed: 1 ins; 1 del; 15 mod; 4400 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
7 lines changed: 0 ins; 3 del; 4 mod; 1290 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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/superword.cpp

rev 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
39 lines changed: 37 ins; 0 del; 2 mod; 1550 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
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/vm_operations.cpp

rev 8995 : 8046155: JEP165: Compiler Control
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 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
40 lines changed: 40 ins; 0 del; 0 mod; 900 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
86 lines changed: 86 ins; 0 del; 0 mod; 616 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
1 line changed: 1 ins; 0 del; 0 mod; 123 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
338 lines changed: 338 ins; 0 del; 0 mod; 0 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
164 lines changed: 164 ins; 0 del; 0 mod; 0 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
743 lines changed: 743 ins; 0 del; 0 mod; 0 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
139 lines changed: 139 ins; 0 del; 0 mod; 0 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
961 lines changed: 961 ins; 0 del; 0 mod; 0 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
130 lines changed: 130 ins; 0 del; 0 mod; 0 unchg

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

rev 8995 : 8046155: JEP165: Compiler Control
Summary:
Reviewed-by:
170 lines changed: 170 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).