[Mesa-dev] Build fail since configure.ac: rework llvm libs handling for 3.9+

[Jan Vesely]

On Mon, 2017-10-16 at 02:55 +0200, Dieter Nützel wrote:
> Am 15.10.2017 01:00, schrieb Andy Furniss:
> > Andy Furniss wrote:
> > > Since
> > > 
> > > commit 13a53c4f5cdd664fd155c9e78fb46a4387af006c
> > > Author: Emil Velikov <emil.velikov at collabora.com>
> > > Date:   Thu Oct 5 11:19:05 2017 +0100
> > > 
> > >      configure.ac: rework llvm libs handling for 3.9+
> > > 
> > > I am getting 00s of
> > > 
> > > .... /mesa/src/amd/common/..... undefined reference to LLVM......
> > > 
> > > Using git llvm have tried with -DLLVM_APPEND_VC_REV=OFF
> > > 
> > > My llvm config =
> > > 
> > > cmake -DCMAKE_INSTALL_PREFIX=/usr -DCMAKE_BUILD_TYPE=Release 
> > > -DLLVM_TARGETS_TO_BUILD="host;AMDGPU" -DLLVM_ENABLE_ASSERTIONS=ON 
> > > -DLLVM_BUILD_LLVM_DYLIB=ON
> > 
> > Build is OK with llvm built with -DLLVM_LINK_LLVM_DYLIB=ON
> 
> Thanks Andy,
> 
> got me much further (LLVM git #a057c40dc32), but now I get this with 
> Clover (--enable-opencl):
> 
> make[4]: Verzeichnis „/opt/mesa/src/gallium/state_trackers/clover“ wird 
> betreten
>    CXX      llvm/codegen/libclllvm_la-common.lo
> llvm/codegen/common.cpp: In function 
> ‘std::vector<clover::module::argument> 
> {anonymous}::make_kernel_args(const llvm::Module&, const string&, const 
> clang::CompilerInstance&)’:
> llvm/codegen/common.cpp:132:52: error: no match for ‘operator-’ (operand 
> types are ‘clang::LangAS’ and ‘const int’)
>                  if (address_space == 
> address_spaces[clang::LangAS::opencl_local
>                                                      
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~
>                                                      - 
> compat::lang_as_offset]) {
>                                                      ^~~~~~~~
> make[4]: *** [Makefile:919: llvm/codegen/libclllvm_la-common.lo] Fehler 
> 1

I think this might be the fallout of clang r315871, which changed
LangAS to a strongly typed enum. casting it to int should work in the
short term, but I think the entire index expression will need to be
moved to compat layer.

Jan

> 
> 
> Without Clover build is OK for me, too.
> 
> Added Jan in CC.
> 
> Dieter

-- 
Jan Vesely <jan.vesely at rutgers.edu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20171015/b9d16138/attachment.sig>