Code Review for hotspot

Prepared by:iignatyev on Mon Jan 25 21:35:46 MSK 2016
Workspace:/Users/iignatye/ws/jdk9/hs-comp-push/hotspot
Compare against: ssh://hg.openjdk.java.net/jdk9/hs-comp/hotspot
Compare against version:9958
Summary of changes: 31 lines changed: 0 ins; 0 del; 31 mod; 3365 unchg
Changeset: hotspot.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/TestCompilerDirectivesCompatibilityBase.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9474 : 8142967: [TESTBUG] Compiler control tests get NullPointerException
Summary: Fix incoorect build jtreg tags
Reviewed-by: iignatyev, neliasso
rev 9148 : 8137167: JEP165: Compiler Control: Implementation task
Summary: Compiler Control JEP
Reviewed-by: roland, twisti, zmajo, simonis
1 line changed: 0 ins; 0 del; 1 mod; 124 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/TestCompilerDirectivesCompatibilityCommandOff.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9474 : 8142967: [TESTBUG] Compiler control tests get NullPointerException
Summary: Fix incoorect build jtreg tags
Reviewed-by: iignatyev, neliasso
rev 9230 : 8141129: 3 compiler control tests fail on product builds
Summary: UnlockDiagnosticVMOptions should be placed before the PrintAssembly
Reviewed-by: kvn
rev 9148 : 8137167: JEP165: Compiler Control: Implementation task
Summary: Compiler Control JEP
Reviewed-by: roland, twisti, zmajo, simonis
1 line changed: 0 ins; 0 del; 1 mod; 80 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/TestCompilerDirectivesCompatibilityCommandOn.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9474 : 8142967: [TESTBUG] Compiler control tests get NullPointerException
Summary: Fix incoorect build jtreg tags
Reviewed-by: iignatyev, neliasso
rev 9230 : 8141129: 3 compiler control tests fail on product builds
Summary: UnlockDiagnosticVMOptions should be placed before the PrintAssembly
Reviewed-by: kvn
rev 9148 : 8137167: JEP165: Compiler Control: Implementation task
Summary: Compiler Control JEP
Reviewed-by: roland, twisti, zmajo, simonis
1 line changed: 0 ins; 0 del; 1 mod; 80 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/TestCompilerDirectivesCompatibilityFlag.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9474 : 8142967: [TESTBUG] Compiler control tests get NullPointerException
Summary: Fix incoorect build jtreg tags
Reviewed-by: iignatyev, neliasso
rev 9230 : 8141129: 3 compiler control tests fail on product builds
Summary: UnlockDiagnosticVMOptions should be placed before the PrintAssembly
Reviewed-by: kvn
rev 9148 : 8137167: JEP165: Compiler Control: Implementation task
Summary: Compiler Control JEP
Reviewed-by: roland, twisti, zmajo, simonis
1 line changed: 0 ins; 0 del; 1 mod; 80 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/commandfile/CompileOnlyTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/commandfile/ExcludeTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9639 : 8144933: CompilerControl: commandfile/ExcludeTest has incorrect jtreg run innotation
Summary: fix incorrect full test name
Reviewed-by: kvn
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/commandfile/LogTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/commandfile/PrintTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/commands/CompileOnlyTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/commands/ExcludeTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/commands/LogTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/commands/PrintTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/directives/CompileOnlyTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 46 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/directives/ExcludeTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/directives/LogTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/directives/PrintTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 45 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/jcmd/AddAndRemoveTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9150 : 8066166: JEP-JDK-8046155: Test task: dcmd tests
Summary: Tests for diagnostic command in CompilerControl
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 77 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/jcmd/AddCompileOnlyTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9150 : 8066166: JEP-JDK-8046155: Test task: dcmd tests
Summary: Tests for diagnostic command in CompilerControl
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 46 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/jcmd/AddExcludeTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9150 : 8066166: JEP-JDK-8046155: Test task: dcmd tests
Summary: Tests for diagnostic command in CompilerControl
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 46 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/jcmd/AddLogTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9150 : 8066166: JEP-JDK-8046155: Test task: dcmd tests
Summary: Tests for diagnostic command in CompilerControl
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 46 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/jcmd/AddPrintAssemblyTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9150 : 8066166: JEP-JDK-8046155: Test task: dcmd tests
Summary: Tests for diagnostic command in CompilerControl
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 47 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/mixed/RandomCommandsTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 43 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/compilercontrol/mixed/RandomValidCommandsTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9638 : 8140667: CompilerControl: tests incorrectly set states for excluded methods
Summary: fix exclude command generation
Reviewed-by: kvn
rev 9217 : 8140668: Quarantine RandomValidCommandsTest
8140669: Quarantine ClearDirectivesFileStackTest
Summary: Quarantine two tests
Reviewed-by: iignatyev, neliasso, kvn
rev 9149 : 8066153: JEP-JDK-8046155: Test task: cover existing
Summary: Tests for CompilerCommand and CompilerControl's directives
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 43 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/cpuflags/TestAESIntrinsicsOnSupportedConfig.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9801 : 8146129: quarantine compiler/cpuflags/TestAESIntrinsicsOnSupportedConfig.java
Reviewed-by: twisti
rev 9660 : 8079667: port vm/compiler/AESIntrinsics/CheckIntrinsics into jtreg
Reviewed-by: kvn
Contributed-by: alexander.vorobyev@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 254 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/cpuflags/TestAESIntrinsicsOnUnsupportedConfig.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9660 : 8079667: port vm/compiler/AESIntrinsics/CheckIntrinsics into jtreg
Reviewed-by: kvn
Contributed-by: alexander.vorobyev@oracle.com
1 line changed: 0 ins; 0 del; 1 mod; 111 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/intrinsics/IntrinsicDisabledTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9218 : 8138651: -XX:DisableIntrinsic matches intrinsics overly eagerly
Summary: Improve parsing of DisableIntrinsic flag.
Reviewed-by: kvn, shade, neliasso
1 line changed: 0 ins; 0 del; 1 mod; 210 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/intrinsics/montgomerymultiply/MontgomeryMultiplyTest.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9213 : 8139907: compiler/intrinsics/montgomerymultiply/MontgomeryMultiplyTest.java fails with timeout
Summary: Change MontgomeryMultiplyTest.java test to execute only on platforms on which the tested intrinsics are available.
Reviewed-by: kvn, neliasso
rev 8619 : 8130150: Implement BigInteger.montgomeryMultiply intrinsic
Summary: Add montgomeryMultiply intrinsics
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 326 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/intrinsics/string/TestStringIntrinsics2.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9958 : 8147937: Adapt SAP copyrights to new company name.
Reviewed-by: simonis, stuefe
rev 9945 : 8145336: PPC64: fix string intrinsics after CompactStrings change
Reviewed-by: mdoerr, aph, kvn
1 line changed: 0 ins; 0 del; 1 mod; 673 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/stable/TestStableMemoryBarrier.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9637 : 8144223: Move j.l.invoke.{ForceInline, DontInline, Stable} to jdk.internal.vm.annotation package
Reviewed-by: jrose, vlivanov, mchung, roland
rev 9270 : 8139758: [REDO] Elide more final field's write memory barrier with escape analysis result
Summary: membar for all final field initializations eliminated if possible
Reviewed-by: roland, vlivanov
1 line changed: 0 ins; 0 del; 1 mod; 92 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/compiler/types/TestMeetIncompatibleInterfaceArrays.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9958 : 8147937: Adapt SAP copyrights to new company name.
Reviewed-by: simonis, stuefe
rev 9431 : 8141551: C2 can not handle returns with inccompatible interface arrays
Reviewed-by: kvn
1 line changed: 0 ins; 0 del; 1 mod; 352 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/gc/g1/TestPLABOutput.java

rev 9959 : 8148012: get rid of slash-dot-dot in @library directives
Reviewed-by:
rev 9764 : 8145092: Use Unified Logging for the GC logging
Summary: JEP-271. VM changes contributed by brutisso, test changes contributed by david.
Reviewed-by: sjohanss, david, brutisso
Contributed-by: bengt.rutisson@oracle.com, david.lindholm@oralce.com
rev 9403 : 8140585: PLAB statistics are flushed too late
Summary: Move the call to G1EvacStats::adjust_desired_plab_sz() to after flushing the per-thread statistics.
Reviewed-by: mgerdin, jmasa
1 line changed: 0 ins; 0 del; 1 mod; 94 unchg

This code review page was prepared using /Users/iignatye/ws/code-tools/webrev/webrev.ksh (vers 25.12-hg+openjdk.java.net).