[Mesa-dev] [PATCH 1/1] gallivm: Fix build after llvm r328839

[Jan Vesely]

On Fri, 2018-03-30 at 18:41 +0200, Dieter Nützel wrote:
> Hello Jan,
> 
> can you have a look on this one (instead) and commit?
> https://lists.freedesktop.org/archives/mesa-dev/2018-March/190535.html
> https://patchwork.freedesktop.org/series/40916/

You should probably ask respective maintainers for that.
Looking at the patch, it'd be nicer if split. I didn't notice that
ac/nir prefixed change touches gallivm. It'd also be preferrable if
LLVM reference used revision numbers instead of git hash (afaik llvm
still uses mainly svn).

Jan

> 
> It has r-b from Samuel Pitoiset and t-b from me.
> 
> Thanks,
> 
> Dieter
> 
> Am 30.03.2018 18:30, schrieb Jan Vesely:
> > LLVM r328839 moved few function declrations to a new Utils.h header.
> > Fixes:
> > gallivm/lp_bld_init.c:145:7: error: implicit declaration of function
> > ‘LLVMAddPromoteMemoryToRegisterPass’; did you mean
> > ‘LLVMAddDemoteMemoryToRegisterPass’?
> > 
> > Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
> > ---
> >  src/gallium/auxiliary/gallivm/lp_bld_init.c | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/src/gallium/auxiliary/gallivm/lp_bld_init.c
> > b/src/gallium/auxiliary/gallivm/lp_bld_init.c
> > index 6ddc509a81..dae9d01552 100644
> > --- a/src/gallium/auxiliary/gallivm/lp_bld_init.c
> > +++ b/src/gallium/auxiliary/gallivm/lp_bld_init.c
> > @@ -40,6 +40,9 @@
> > 
> >  #include <llvm-c/Analysis.h>
> >  #include <llvm-c/Transforms/Scalar.h>
> > +#if HAVE_LLVM >= 0x0700
> > +#include <llvm-c/Transforms/Utils.h>
> > +#endif
> >  #include <llvm-c/BitWriter.h>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180330/ea995766/attachment.sig>