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

[Igor Ignatyev]

Hi Chris,

What about regression test for this? As I understood from your 
comment[1], you have it, but the webrev and changeset don't contain it.

[1] 
https://bugs.openjdk.java.net/browse/JDK-8022395?focusedCommentId=13447393&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13447393
Igor

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.
>>
>> igor
>>
>> On Jan 16, 2014, at 1:49 PM, Christian Thalinger
>> <christian.thalinger at oracle.com
>> <mailto:christian.thalinger at oracle.com>> 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.
>>>
>>
>