Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-23 Thread Mauro Rossi
Il 23/ago/2017 19:57, "Rob Herring" ha scritto: On Wed, Aug 23, 2017 at 12:31 PM, Emil Velikov wrote: > On 23 August 2017 at 17:50, Rob Herring wrote: >> On Sun, Aug 20, 2017 at 2:57 PM, Rob Herring wrote: >>> On Fri, Aug 18, 2017 at 8:53 PM, Chih-Wei Huang wrote: 2017-08-19 8:27 GMT+08:

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-23 Thread Rob Herring
On Wed, Aug 23, 2017 at 12:31 PM, Emil Velikov wrote: > On 23 August 2017 at 17:50, Rob Herring wrote: >> On Sun, Aug 20, 2017 at 2:57 PM, Rob Herring wrote: >>> On Fri, Aug 18, 2017 at 8:53 PM, Chih-Wei Huang >>> wrote: 2017-08-19 8:27 GMT+08:00 Emil Velikov : > On 18 August 2017 at

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-23 Thread Emil Velikov
On 23 August 2017 at 17:50, Rob Herring wrote: > On Sun, Aug 20, 2017 at 2:57 PM, Rob Herring wrote: >> On Fri, Aug 18, 2017 at 8:53 PM, Chih-Wei Huang >> wrote: >>> 2017-08-19 8:27 GMT+08:00 Emil Velikov : On 18 August 2017 at 20:46, Rob Herring wrote: > Both statically linking libLL

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-23 Thread Rob Herring
On Sun, Aug 20, 2017 at 2:57 PM, Rob Herring wrote: > On Fri, Aug 18, 2017 at 8:53 PM, Chih-Wei Huang > wrote: >> 2017-08-19 8:27 GMT+08:00 Emil Velikov : >>> On 18 August 2017 at 20:46, Rob Herring wrote: Both statically linking libLLVMCore and dynamically linking libLLVM causes dupl

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-21 Thread Mauro Rossi
2017-08-21 23:57 GMT+02:00 Rob Herring : > On Mon, Aug 21, 2017 at 4:44 PM, Mauro Rossi wrote: >> 2017-08-18 21:46 GMT+02:00 Rob Herring : >>> Both statically linking libLLVMCore and dynamically linking libLLVM causes >>> duplicated symbols in gallium_dri.so and it fails to dlopen. We don't >>> re

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-21 Thread Rob Herring
On Mon, Aug 21, 2017 at 4:44 PM, Mauro Rossi wrote: > 2017-08-18 21:46 GMT+02:00 Rob Herring : >> Both statically linking libLLVMCore and dynamically linking libLLVM causes >> duplicated symbols in gallium_dri.so and it fails to dlopen. We don't >> really need to link libLLVMCore, but just need ge

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-21 Thread Mauro Rossi
2017-08-18 21:46 GMT+02:00 Rob Herring : > Both statically linking libLLVMCore and dynamically linking libLLVM causes > duplicated symbols in gallium_dri.so and it fails to dlopen. We don't > really need to link libLLVMCore, but just need generated headers to be > built first. Dynamically linking t

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-21 Thread Rob Herring
On Mon, Aug 21, 2017 at 12:47 PM, Emil Velikov wrote: > On 21 August 2017 at 17:17, Rob Herring wrote: >> On Fri, Aug 18, 2017 at 7:27 PM, Emil Velikov >> wrote: >>> On 18 August 2017 at 20:46, Rob Herring wrote: Both statically linking libLLVMCore and dynamically linking libLLVM causes >

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-21 Thread Emil Velikov
On 21 August 2017 at 17:17, Rob Herring wrote: > On Fri, Aug 18, 2017 at 7:27 PM, Emil Velikov > wrote: >> On 18 August 2017 at 20:46, Rob Herring wrote: >>> Both statically linking libLLVMCore and dynamically linking libLLVM causes >>> duplicated symbols in gallium_dri.so and it fails to dlope

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-21 Thread Rob Herring
On Fri, Aug 18, 2017 at 7:27 PM, Emil Velikov wrote: > On 18 August 2017 at 20:46, Rob Herring wrote: >> Both statically linking libLLVMCore and dynamically linking libLLVM causes >> duplicated symbols in gallium_dri.so and it fails to dlopen. We don't >> really need to link libLLVMCore, but just

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-21 Thread Emil Velikov
On 20 August 2017 at 20:57, Rob Herring wrote: > > What does either really buy us? It's really just bike shedding and > unrelated to fixing the problem at hand. > Agreed - it's orthogonal to the issue at hand, I should have made this clearer. It's an open question, which is not meant to hold the

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-20 Thread Rob Herring
On Fri, Aug 18, 2017 at 8:53 PM, Chih-Wei Huang wrote: > 2017-08-19 8:27 GMT+08:00 Emil Velikov : >> On 18 August 2017 at 20:46, Rob Herring wrote: >>> Both statically linking libLLVMCore and dynamically linking libLLVM causes >>> duplicated symbols in gallium_dri.so and it fails to dlopen. We do

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-18 Thread Yu, Qiang
> gallium_dri_la_LDFLAGS += \ > -Wl,--version-script=$(top_srcdir)/src/gallium/targets/dri/dri.sym A nice trick to know about. Regards, Qiang ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-18 Thread Chih-Wei Huang
2017-08-19 8:27 GMT+08:00 Emil Velikov : > On 18 August 2017 at 20:46, Rob Herring wrote: >> Both statically linking libLLVMCore and dynamically linking libLLVM causes >> duplicated symbols in gallium_dri.so and it fails to dlopen. We don't >> really need to link libLLVMCore, but just need generat

Re: [Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-18 Thread Emil Velikov
On 18 August 2017 at 20:46, Rob Herring wrote: > Both statically linking libLLVMCore and dynamically linking libLLVM causes > duplicated symbols in gallium_dri.so and it fails to dlopen. We don't > really need to link libLLVMCore, but just need generated headers to be > built first. Dynamically li

[Mesa-dev] [PATCH] Android: Fix LLVM duplicated symbols linking for N and M

2017-08-18 Thread Rob Herring
Both statically linking libLLVMCore and dynamically linking libLLVM causes duplicated symbols in gallium_dri.so and it fails to dlopen. We don't really need to link libLLVMCore, but just need generated headers to be built first. Dynamically linking to libLLVM instead is enough to do that. Thanks to