[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