[8] RFR (XXS): 8022395: java.util.zip.ZipException: Not in GZIP format in JT_JDK/test/java/util/zip/GZIP tests

Vladimir Kozlov vladimir.kozlov at oracle.com
Thu Jan 16 14:13:01 PST 2014


Looks good.

Thanks,
Vladimir

On 1/16/14 1:49 PM, Christian Thalinger wrote:
> https://bugs.openjdk.java.net/browse/JDK-8022395
> http://cr.openjdk.java.net/~twisti/8022395/webrev.00
>
> 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.
>


More information about the hotspot-compiler-dev mailing list