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

[Coleen Phillimore]

Ok, I checked it in with author sgehwolf, through JPRT although JPRT 
doesn't build it.
Thanks for the contribution Severin.
Coleen

On 1/16/16 5:43 AM, Volker Simonis wrote:
> Hi Coleen,
>
> I totally agree and I'd like to but there have been complaints before 
> when we pushed ppc64 or AIX only changes to shared directories.
>
> So until we don't get a general and clear agreement on this topic I'd 
> like to ask you to be so kind and do the push.
>
> Thank you and best regards,
> Volker
>
> On Friday, January 15, 2016, Coleen Phillimore 
> <coleen.phillimore at oracle.com <mailto:coleen.phillimore at oracle.com>> 
> wrote:
>
>
>
>     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/
>             <http://cr.openjdk.java.net/%7Esgehwolf/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
>
>