hotspot (original) (raw)

Code Review for hotspot

Prepared by: roland on Fri Jun 12 14:25:10 CEST 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: currenthead
Summary of changes: 160 lines changed: 109 ins; 37 del; 14 mod; 7536 unchg
Patch of changes: <hotspot.patch>
Legend: Modified file Deleted file New file

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

rev 8568 : 8086046: escape analysis generates incorrect code as of B67 Summary: load bypasses arraycopy that sets the value after the ArrayCopyNode is expanded Reviewed-by:

57 lines changed: 26 ins; 31 del; 0 mod; 1924 unchg

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

rev 8568 : 8086046: escape analysis generates incorrect code as of B67 Summary: load bypasses arraycopy that sets the value after the ArrayCopyNode is expanded Reviewed-by:

15 lines changed: 6 ins; 5 del; 4 mod; 1082 unchg

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

rev 8052 : castpp gcm rev 8568 : 8086046: escape analysis generates incorrect code as of B67 Summary: load bypasses arraycopy that sets the value after the ArrayCopyNode is expanded Reviewed-by:

17 lines changed: 6 ins; 1 del; 10 mod; 4530 unchg

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

rev 8568 : 8086046: escape analysis generates incorrect code as of B67 Summary: load bypasses arraycopy that sets the value after the ArrayCopyNode is expanded Reviewed-by:

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


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