hotspot (original) (raw)

Code Review for hotspot

Prepared by: roland on Thu Mar 12 16:58:46 CET 2015
Workspace: /net/mackdrive/export/jrts/work/rw140007/jdk9-repo/hotspot
Compare against: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot
Compare against version: 8000
Summary of changes: 303 lines changed: 226 ins; 53 del; 24 mod; 11510 unchg
Patch of changes: <hotspot.patch>
Legend: Modified file Deleted file New file

[Cdiffs](src/share/vm/opto/graphKit.cpp.cdiff.html) [Udiffs](src/share/vm/opto/graphKit.cpp.udiff.html) [Sdiffs](src/share/vm/opto/graphKit.cpp.sdiff.html) [Frames](src/share/vm/opto/graphKit.cpp.frames.html) [Old](src/share/vm/opto/graphKit.cpp-.html) [New](src/share/vm/opto/graphKit.cpp.html) [Patch](src/share/vm/opto/graphKit.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/graphKit.cpp) src/share/vm/opto/graphKit.cpp

rev 8008 : 8073866: Fix for 8064703 is not sufficient Summary: side effects between allocation and arraycopy can be reexecuted, unreachable uninitialized array can be seen by GCs Reviewed-by:

12 lines changed: 0 ins; 8 del; 4 mod; 4342 unchg

[Cdiffs](src/share/vm/opto/graphKit.hpp.cdiff.html) [Udiffs](src/share/vm/opto/graphKit.hpp.udiff.html) [Sdiffs](src/share/vm/opto/graphKit.hpp.sdiff.html) [Frames](src/share/vm/opto/graphKit.hpp.frames.html) [Old](src/share/vm/opto/graphKit.hpp-.html) [New](src/share/vm/opto/graphKit.hpp.html) [Patch](src/share/vm/opto/graphKit.hpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/graphKit.hpp) src/share/vm/opto/graphKit.hpp

rev 8008 : 8073866: Fix for 8064703 is not sufficient Summary: side effects between allocation and arraycopy can be reexecuted, unreachable uninitialized array can be seen by GCs Reviewed-by:

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

[Cdiffs](src/share/vm/opto/library%5Fcall.cpp.cdiff.html) [Udiffs](src/share/vm/opto/library%5Fcall.cpp.udiff.html) [Sdiffs](src/share/vm/opto/library%5Fcall.cpp.sdiff.html) [Frames](src/share/vm/opto/library%5Fcall.cpp.frames.html) [Old](src/share/vm/opto/library%5Fcall.cpp-.html) [New](src/share/vm/opto/library%5Fcall.cpp.html) [Patch](src/share/vm/opto/library%5Fcall.cpp.patch) [Raw](raw%5Ffiles/new/src/share/vm/opto/library%5Fcall.cpp) src/share/vm/opto/library_call.cpp

rev 8008 : 8073866: Fix for 8064703 is not sufficient Summary: side effects between allocation and arraycopy can be reexecuted, unreachable uninitialized array can be seen by GCs Reviewed-by:

218 lines changed: 158 ins; 44 del; 16 mod; 5830 unchg

[Cdiffs](test/compiler/arraycopy/TestArrayCopyNoInit.java.cdiff.html) [Udiffs](test/compiler/arraycopy/TestArrayCopyNoInit.java.udiff.html) [Sdiffs](test/compiler/arraycopy/TestArrayCopyNoInit.java.sdiff.html) [Frames](test/compiler/arraycopy/TestArrayCopyNoInit.java.frames.html) [Old](test/compiler/arraycopy/TestArrayCopyNoInit.java-.html) [New](test/compiler/arraycopy/TestArrayCopyNoInit.java.html) [Patch](test/compiler/arraycopy/TestArrayCopyNoInit.java.patch) [Raw](raw%5Ffiles/new/test/compiler/arraycopy/TestArrayCopyNoInit.java) test/compiler/arraycopy/TestArrayCopyNoInit.java

rev 8008 : 8073866: Fix for 8064703 is not sufficient Summary: side effects between allocation and arraycopy can be reexecuted, unreachable uninitialized array can be seen by GCs Reviewed-by:

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

[Cdiffs](test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java.cdiff.html) [Udiffs](test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java.udiff.html) [Sdiffs](test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java.sdiff.html) [Frames](test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java.frames.html) [Old](test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java-.html) [New](test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java.html) [Patch](test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java.patch) [Raw](raw%5Ffiles/new/test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java) test/compiler/arraycopy/TestArrayCopyNoInitDeopt.java

rev 8008 : 8073866: Fix for 8064703 is not sufficient Summary: side effects between allocation and arraycopy can be reexecuted, unreachable uninitialized array can be seen by GCs Reviewed-by:

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

------ ------ ------ ------ ---[New](test/compiler/arraycopy/TestArrayCopyBadReexec.java.html) [Patch](test/compiler/arraycopy/TestArrayCopyBadReexec.java.patch) [Raw](raw%5Ffiles/new/test/compiler/arraycopy/TestArrayCopyBadReexec.java) test/compiler/arraycopy/TestArrayCopyBadReexec.java

rev 8008 : 8073866: Fix for 8064703 is not sufficient Summary: side effects between allocation and arraycopy can be reexecuted, unreachable uninitialized array can be seen by GCs Reviewed-by:

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


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