[9] RFR(XS): 8147482: Zero build fails after 8144953

Coleen Phillimore coleen.phillimore at oracle.com
Fri Jan 15 19:52:08 UTC 2016



On 1/15/16 2:20 PM, Volker Simonis wrote:
> Hi Severin,
>
> the change looks good.
>
> Unfortunately I can't sponsor it because it is in shared code.

Actually, since it's in Zero only, I think it would be okay to push 
directly.   It's not technically in the cpu/zero/vm directory but these 
files are only used for Zero.

Alas, we don't test Zero with JPRT.  If we start doing this, a. it 
wouldn't break so often and b. we'd have to sponsor just to make sure it 
doesn't break and prevent other integration jobs from going through.

Thanks,
Coleen
>
> Regards,
> Volker
>
>
> On Fri, Jan 15, 2016 at 7:46 PM, Severin Gehwolf <sgehwolf at redhat.com> wrote:
>> Hi,
>>
>> Could somebody please review and sponsor this trivial change to
>> bytecodeInterpreter.cpp. It fixes a Zero-only build problem.
>>
>> Webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8147482/webrev.01/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8147482
>>
>> Zero builds fine in release/fastdebug/slowdebug variants after this.
>>
>> Thanks,
>> Severin



More information about the hotspot-dev mailing list