Code Review for hotspot

Prepared by:twisti on Thu Jan 16 13:47:59 PST 2014
Workspace:/Users/twisti/ws/jdk9/hs-comp/hotspot
Compare against: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot
Summary of changes: 2 lines changed: 2 ins; 0 del; 0 mod; 1488 unchg
Patch of changes: hotspot.patch
Author comments:
8022395: java.util.zip.ZipException: Not in GZIP format in JT_JDK/test/java/util/zip/GZIP tests
Reviewed-by:

C1's instrinsic for java.util.zip.CRC32.update(int b) on x86 destroys the input value b but the register is not marked to be destroyed. This can lead to a wrong value if CRC32.update is inlined.

The fix is to mark the LIRItem to be destroyed. This will make C1 to insert a spill move.

Bug id: 8022395
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/cpu/x86/vm/c1_LIRGenerator_x86.cpp

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

This code review page was prepared using /Users/twisti/bin/webrev (vers 23.18-hg-never-dev).