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

rev 12726 : 8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
Summary: replacing load on dst with load on src only valid if copy doesn't modify src element to load
Reviewed-by:
rev 12727 : review

70 lines changed: 47 ins; 3 del; 20 mod; 690 unchg

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

rev 12726 : 8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
Summary: replacing load on dst with load on src only valid if copy doesn't modify src element to load
Reviewed-by:

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

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

rev 12726 : 8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
Summary: replacing load on dst with load on src only valid if copy doesn't modify src element to load
Reviewed-by:

9 lines changed: 7 ins; 0 del; 2 mod; 4511 unchg

------ ------ ------ ------ --- New Patch Raw test/compiler/arraycopy/TestACSameSrcDst.java

rev 12726 : 8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
Summary: replacing load on dst with load on src only valid if copy doesn't modify src element to load
Reviewed-by:

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

This code review page was prepared using /home/rwestrel/code-tools/webrev/webrev.ksh (vers 25.16-hg+openjdk.java.net).