[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-jdk7u-dev
Subject:    Re: [7u-dev] Request for approval for CR 8005722
From:       BILL PITTORE <bill.pittore () oracle ! com>
Date:       2013-02-28 15:37:09
Message-ID: 512F79A5.7090101 () oracle ! com
[Download RAW message or body]

Hmm, ok. I do recall having to send out a request last time I wanted to 
push into jdk7u-dev. Did I muddy up the waters by mentioning hs24 in my 
email?  So if I push into hs24, no approval needed. Push into jdk7u-dev, 
approval needed; correct?

bill

On 2/27/2013 11:29 PM, David Holmes wrote:
> Hi Bill,
>
> On 28/02/2013 2:04 AM, BILL PITTORE wrote:
>> Would like approval to backport the fix for 8005722 into hs24. Fix is
>> identical to hs25.
>
> Fixes to hs24 don't need to get individual approval this way. 
> Alejandro requests a bulk integration approval when pushing hs24 to 7u.
>
> You can simply push this to hs24
>
> David
> -----
>
>> Bug report: http://bugs.sun.com/view_bug.do?bug_id=8005722
>>
>> Summary: Assert triggered in c1_LIR.cpp
>>
>> assert(_oprs_len[mode] < maxNumberOfOperands, "array overflow")
>>
>> The problem is not the assert but the fact that in
>> LIR_OpVisitState::visit()
>>   the receiver operand is processed twice.
>>
>> Reviewed-by: roland, vladidan
>>
>> http://cr.openjdk.java.net/~bpittore/8005722/webrev.01
>>
>> thanks,
>> bill
>>


[prev in list] [next in list] [prev in thread] [next in thread]