[Mesa-dev] [PATCH 2/2] clover: Fix building with latest llvm

[Bruno Jimenez]

It's strange, I have just compiled with the latest llvm revision at
Arch's mesa-git repo (202852) without problems.

I'm not an automake expert either, but at gallium/auxiliary/Makefile.am
there's this:

AM_CXXFLAGS += \
	$(GALLIUM_CFLAGS) \
	$(LLVM_CXXFLAGS)

So I suppose, of course I may be wrong, that it is using the correct
flags for llvm.

How does it fail to compile? does it complains about '-std=X' or other
flags?

Bruno

On Wed, 2014-03-05 at 00:43 +0100, Marek Olšák wrote:
> I'm not an automake expect, but this might need to be done for gallivm
> too. It currently fails to compile with latest LLVM.
> 
> Marek
> 
> On Sun, Mar 2, 2014 at 10:39 PM, Bruno Jiménez <brunojimen at gmail.com> wrote:
> > ---
> >  src/gallium/state_trackers/clover/Makefile.am | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/src/gallium/state_trackers/clover/Makefile.am b/src/gallium/state_trackers/clover/Makefile.am
> > index ece2b38..75eae86 100644
> > --- a/src/gallium/state_trackers/clover/Makefile.am
> > +++ b/src/gallium/state_trackers/clover/Makefile.am
> > @@ -37,9 +37,8 @@ libcltgsi_la_SOURCES = \
> >         tgsi/compiler.cpp
> >
> >  libclllvm_la_CXXFLAGS = \
> > -       -std=c++98 \
> >         $(VISIBILITY_CXXFLAGS) \
> > -       $(LLVM_CPPFLAGS) \
> > +       $(LLVM_CXXFLAGS) \
> >         $(DEFINES) \
> >         -DLIBCLC_INCLUDEDIR=\"$(LIBCLC_INCLUDEDIR)/\" \
> >         -DLIBCLC_LIBEXECDIR=\"$(LIBCLC_LIBEXECDIR)/\" \
> > @@ -49,7 +48,7 @@ libclllvm_la_SOURCES = \
> >         llvm/invocation.cpp
> >
> >  libclover_la_CXXFLAGS = \
> > -       -std=c++0x \
> > +       -std=c++11 \
> >         $(VISIBILITY_CXXFLAGS)
> >
> >  libclover_la_LIBADD = \
> > --
> > 1.9.0
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev