OpenJDK 8 and Shark

[Severin Gehwolf]

Hi,

cc'ing Roman as he might know what llvm version works/worked for shark
pending some additional fixes.

On Wed, 2015-01-07 at 13:08 +0100, Mario Torre wrote:
> On Mon, 2015-01-05 at 11:08 +0000, Andrew Haley wrote:
> > > 
> > > 2) What was the last version of LLVM that anyone used to build
> > Shark?  The 
> > > latest gcc compiler complains about "#elif _MSC_VER" not being
> > defined in the 
> > > llvm 3.4 headers.  Changing it to "#elif defined(_MSC_VER)" solves
> > the problem.
> 
> I think I pushed a patch to address this on JDK9, I remember it was
> backported to 8 also, so Shark on 8 and 9 "should" be buildable with
> llvm 3.4 and 3.2.
> 
> I think the patch was here (IIRC):
> 
> http://cr.openjdk.java.net/~neugens/zero-2014-02-24/webrev.03/
> 
> I'm CC'ing Severin that took over most of my work there, maybe he knows
> something more.

Sorry, this does not ring a bell for me. The last time I've built shark
was a while ago (> 6 months back) and I think I've used llvm 3.4. I may
have used the above patches, not sure. Last I recall was that shark
debug builds were terribly broken. Shark release builds somewhat worked,
after you got it to compile but still had many bugs. I've tried on
x86_64 only.

> It's possible something changed in the JDK so that this patch is not
> valid anymore, unfortunately as Andrew says LLVM is too much of a moving
> target to keep up with it, but any patches are welcomed!

Speaking of Zero alone I know that upstream OpenJDK changes have been
breaking the zero variant build a couple of times. Thus, I would not be
surprised if shark is broken in a similar way. In general, I try to fix
Zero compile issues as soon as they get noticed via:
http://builder.classpath.org/jenkins/job/OpenJDK9_hscomp_Zero/

Cheers,
Severin