[Beignet] beignet doesnt compile with LLVM 3.6

[Zou, Nanhai]

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Zhigang Gong
> Sent: Wednesday, February 11, 2015 12:37 PM
> To: Steven Newbury
> Cc: junyan.he at linux.intel.com; Paulo Dias; Yang, Rong R;
> beignet at lists.freedesktop.org
> Subject: Re: [Beignet] beignet doesnt compile with LLVM 3.6
> 
> On Tue, Feb 10, 2015 at 02:47:56PM +0000, Steven Newbury wrote:
> >
> > On Thu, 2014-10-30 at 02:22 +0000, Yang, Rong R wrote:
> > > Because LLVM change the head file frequently, Beignet only support
> > > the released version LLVM, from 3.3 to 3.5. You can get this
> > > information from Beignet wiki:
> > > http://www.freedesktop.org/wiki/Software/Beignet/.
> > >
> >
> >
> >
> > This is just a suggestion; but wouldn't it be better to have Beignet
> > master track the upstream "stable" LLVM branch, ie. currently 3.6 pre-
> > release, and have release versions of Beignet target the currently
> > released LLVM version with bugfix, and possibly stable feature updates?
> >
> > This way Beignet isn't always playing catch up when new LLVM versions
> > appear and doesn't cause distributors issues with multiple LLVM
> > version requirements between Mesa and Beignet.
> >
> > Downside, of course, is Beignet developers would have to change their
> > development process to include building of the branched "stable" LLVM
> > when working on master branch to ensure no breakage occurs.
> >
> 
> This is a good suggestion. But it may not be easy for us to fully follow.
> As you may know, upgrading to a new LLVM version always causes some issues
> and some of them are not easy to fix. Considering we have the following to do
> items:
> 
> 1. Enable BDW
> 2. Enable SKL
> 3. Bug fixing
> 4. Performance tuning
> 6. support LLVM 3.6 pre-release branch fix all the issues.
> 
> Currently, the 6th item will have lower priority than others. Once we get the
> official LLVM 3.6 released, we will raise its priority and make sure the master
> branch and the next beignet release support the latest official LLVM version.
> This style will minimum the effort of catching up LLVM version change.
> Considering LLVM 3.6 will be officially released at 21 February. As we will have
> Chinese New Year vacation then, we will provide LLVM 3.6 support in the
> middle of March.
> 
> But I think things will become better in the near future once we have all
> platforms enabled and we have less bugs then :) or we can get more support
> from the community.
> 
> Then we can fully follow your suggestion to catch up the LLVM stable branch
> not just the released version. Hope we can switch to that style for next LLVM
> major release.
> 
> Thanks,
> Zhigang Gong.
> 
Hi,

We need to spend quite some time for QA testing and performance evaluation for every upgrade with a new llvm version to make sure Beignet release
has the quality and performance we expected.

To support 2 or 3 version of llvm at the same time will add a lot of additional testing effort for us.

So I prefer to track the llvm stable branch instead of llvm releases at this point. 

Thanks
Zou Nanhai

> > _______________________________________________
> > Beignet mailing list
> > Beignet at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/beignet
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet