[Mesa-dev] [PATCH] configure.ac: Link to libLLVMCodegen to fix cyclic linking problems

[Stefan Dirsch]

B41;308;0cOn Tue, Feb 27, 2018 at 04:24:04PM +0100, Stefan Dirsch wrote:
> On Tue, Feb 27, 2018 at 11:59:22AM +0000, Emil Velikov wrote:
> > On 23 February 2018 at 15:02, Stefan Dirsch <sndirsch at suse.de> wrote:
> > > Patch by "Ismail Doenmez" <idonmez at suse.com>
> > > https://bugzilla.suse.com/show_bug.cgi?id=1082307
> > 
> > Can anyone provide some information about the cyclic linking problem?
> > Stuff like - mesa configure options, LLVM version and/or actual error
> > will be appreciated.
> 
> Good question. I believe we no longer need this patch since we use
> LLVM_BUILD_LLVM_DYLIB instead of BUILD_SHARED_LIBS to build one single libLLVM
> library, which is the recommended way anyway.
> 
> Michal, is this correct? Could we drop this patch?

Michal meanwhile confirmed.

"Exactly. We do not need it anymore since that change in LLVM build. It would
be still useful for anyone who uses Mesa with LLVM build using
BUILD_SHARED_LIBS, but it is just workaround for issue in LLVM. Which LLVM
upstream does not want to address since the build mode is not meant for this
use case."

I'm removing the patch from (open)SUSE packages. ;-)

Thanks,
Stefan

Public Key available
------------------------------------------------------
Stefan Dirsch (Res. & Dev.)   SUSE LINUX GmbH
Tel: 0911-740 53 0            Maxfeldstraße 5
FAX: 0911-740 53 479          D-90409 Nürnberg
http://www.suse.de            Germany 
---------------------------------------------------------------
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham
Norton, HRB 21284 (AG Nürnberg)
---------------------------------------------------------------