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
rev 12728 : more review + new tests & fixes
rev 12729 : -XX:-ReduceInitialCardMarks fix

125 lines changed: 79 ins; 5 del; 41 mod; 667 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:
rev 12728 : more review + new tests & fixes

3 lines changed: 1 ins; 0 del; 2 mod; 175 unchg

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

rev 12569 : 8173147: [ctw] fails during compilation of sun.security.krb5.internal.crypto.RsaMd5DesCksumType::calculateKeyedChecksum with " graph should be schedulable"
Summary: Loads generated at uncommon trap from eliminated arraycopy have incorrect memory state
Reviewed-by: thartmann
rev 12697 : 8176506: C2: loop unswitching and unsafe accesses cause crash
Reviewed-by:
rev 12728 : more review + new tests & fixes

3 lines changed: 2 ins; 0 del; 1 mod; 2789 unchg

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

rev 12728 : more review + new tests & fixes
rev 12729 : -XX:-ReduceInitialCardMarks fix
rev 12730 : lost mem dependency

36 lines changed: 29 ins; 5 del; 2 mod; 1260 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:
rev 12728 : more review + new tests & fixes
rev 12730 : lost mem dependency

106 lines changed: 106 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).