[9] RFR(XS) 8141138: Zero fails to build

[Severin Gehwolf]

Hi Coleen,

On Mon, 2015-11-02 at 13:05 -0500, Coleen Phillimore wrote:
> Hi,
> 
> This change looks fine.

Thanks for the review.

> I can't build Zero anymore because of some 
> build change that doesn't work for me.  If I can get that figured
> out, 
> I'll sponsor this and the other change for you.

Oh that's unfortunate. Please let me know if there is something I can
do to fix it.

> The build tries to pass -lffi even though I gave it an explicit
> libffi 
> directory.

I see. Just to clarify: you've used "--with-libffi" or "--with-libffi-
include and --with-libffi-lib", right? I have libffi in a well known
system location (package libffi on Fedora) which does not seem to need
this.

Cheers,
Severin

> Coleen
> 
> On 11/2/15 11:41 AM, Severin Gehwolf wrote:
> > Hi,
> > 
> > Could somebody please review and - if approved - sponsor this
> > fairly
> > trivial change?
> > 
> > A recent checkout of the hs-rt tree fails to build the Zero
> > variant[1]
> > for the following reasons:
> > 
> >   * JDK-8136421 (Java Level JVM Compiler Interface) did not update
> > two
> >     files: src/cpu/zero/vm/relocInfo_zero.cpp
> >     and src/cpu/zero/vm/compiledIC_zero.cpp. The signature
> >     of emit_to_interp_stub() has changed. This patch updates it for
> > Zero
> >     too. poll_return_Relocation::fix_relocation_after_move has been
> >     removed from header files. This patch removes the Zero stub as
> > well.
> >   * JDK-8139163 (InstanceKlass::cast passes through NULL) removed
> > the
> >     k_entry variable. I've updated the code to use the new "ik"
> > variable
> >     instead.
> > 
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8141138
> > webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8141138/we
> > brev.00/
> > 
> > Thanks,
> > Severin
> > 
> > [1] http://builder.classpath.org/jenkins/job/OpenJDK9_hs_rt_Zero/18
> > 2/
>