RFR (XXS): 8022395: java.util.zip.ZipException: Not in GZIP format in JT_JDK/test/java/util/zip/GZIP tests
christian.thalinger at oracle.com
Fri Jan 17 13:21:35 PST 2014
The bug was found with an already existing regression test in the JDK. I don’t think we have to duplicate that test somewhere else.
On Jan 17, 2014, at 1:04 AM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> Hi Chris,
> What about regression test for this? As I understood from your comment, you have it, but the webrev and changeset don't contain it.
>  https://bugs.openjdk.java.net/browse/JDK-8022395?focusedCommentId=13447393&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13447393
> On 01/17/2014 04:25 AM, Christian Thalinger wrote:
>> Thank you, Vladimir and Igor. I’m going to push this to jdk9 too.
>> On Jan 16, 2014, at 3:48 PM, Igor Veresov <igor.veresov at oracle.com
>> <mailto:igor.veresov at oracle.com>> wrote:
>>> Looks fine.
>>> On Jan 16, 2014, at 1:49 PM, Christian Thalinger
>>> <christian.thalinger at oracle.com
>>> <mailto:christian.thalinger at oracle.com>> wrote:
>>>> 8022395: java.util.zip.ZipException: Not in GZIP format in
>>>> JT_JDK/test/java/util/zip/GZIP tests
>>>> 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