On 07/01/13 10:52, ext Pekka Jääskeläinen wrote:
> On 01/07/2013 12:16 AM, Vincent Danjean wrote:

>> - the second patch try to remove superfluous linkage for various
>>     libraries and binaries. These have been detected by debian build
>>     tools (libraries and binaries link to libraries without using
>>     any of there symbols).
>>     Note that at several places, "llvm-config --ldflags" was used
>>     instead of linking to libllvm. "llvm-config --ldflags" are the
>>     flags used to build libllvm itself, not what you want to use
>>     when to want to link *to* libllvm.
>>     Note also that I only checked/looked at what is currently used
>>     in the Debian package (ie not tce, ...)
>
> Also this. I'll fix the possible issues with TCE separately.
>

I partially reverted this in 511, as it broke the build on all my 
machines (most of which are debian, btw ;) ), as libLLVM.so "is not found".

What was the original issue with having these flags in in the first place?

For me, 'llvm-config --ldflags' prints out:
  '-L/opt/llvm/3.1/lib  -lpthread -ldl -lm'

I.e. the flags to pass to the linker. Why it also links in math, 
pthreads and dl - I don't know.


kalle

-- 
But beware the debugger. Dark side of the source it is.
If once you start down the dark path, forever will it dominate
your destiny. Consume you it will.

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. SALE $99.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122412
_______________________________________________
pocl-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/pocl-devel

Reply via email to