<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Looks fine.<div><br></div><div>igor</div><div><br><div style=""><div>On Jan 16, 2014, at 1:49 PM, Christian Thalinger &lt;<a href="mailto:christian.thalinger@oracle.com">christian.thalinger@oracle.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=us-ascii"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div style="margin: 0px; font-size: 10px; font-family: Monaco;"><a href="https://bugs.openjdk.java.net/browse/JDK-8022395">https://bugs.openjdk.java.net/browse/JDK-8022395</a></div><div style="margin: 0px; font-size: 10px; font-family: Monaco;"><a href="http://cr.openjdk.java.net/~twisti/8022395/webrev.00">http://cr.openjdk.java.net/~twisti/8022395/webrev.00</a></div><div style="margin: 0px; font-size: 10px; font-family: Monaco; min-height: 14px;"><br></div><div style="margin: 0px; font-size: 10px; font-family: Monaco;">8022395: java.util.zip.ZipException: Not in GZIP format in JT_JDK/test/java/util/zip/GZIP tests</div><div style="margin: 0px; font-size: 10px; font-family: Monaco;">Reviewed-by:</div><div style="margin: 0px; font-size: 10px; font-family: Monaco; min-height: 14px;"><br></div><div style="margin: 0px; font-size: 10px; font-family: Monaco;">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.&nbsp; This can lead to a wrong value if CRC32.update is inlined.</div><div style="margin: 0px; font-size: 10px; font-family: Monaco; min-height: 14px;"><br></div><div style="margin: 0px; font-size: 10px; font-family: Monaco;">The fix is to mark the LIRItem to be destroyed.&nbsp; This will make C1 to insert a spill move.</div><div><br></div></div></blockquote></div><br></div></body></html>