Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linking to LLVMOProfileJIT #121

Closed
pwuertz opened this issue Feb 18, 2016 · 8 comments
Closed

Linking to LLVMOProfileJIT #121

pwuertz opened this issue Feb 18, 2016 · 8 comments

Comments

@pwuertz
Copy link

pwuertz commented Feb 18, 2016

Compiling the current llvmlite on Ubuntu 15.10 using the llvm 3.7 packages from the standard repository fails due to a hardcoded linkage to LLVMOProfileJIT. This library doesn't exist on the system. After removing the link from Makefile.linux llvmlite builds and runs without problems.

@mschubert
Copy link

same on Archlinux

@navreetaft
Copy link

@pitrou
Copy link
Contributor

pitrou commented Mar 2, 2016

Also see previous reports at #100 (comment)

@pitrou
Copy link
Contributor

pitrou commented Mar 2, 2016

Also, upstream issue (explaining the hardcoded linkage) at https://llvm.org/bugs/show_bug.cgi?id=25091

@pwuertz
Copy link
Author

pwuertz commented Mar 2, 2016

Is there a possible workaround for determining if llvm is built with oprofile support or not?
@pitrou: Does oprofile show up in llvm-config --components for you?
Maybe enable the flag after checking for LLVMOProfileJIT files in llvm-config --libdir?

@pitrou
Copy link
Contributor

pitrou commented Mar 2, 2016

Yes, oprofilejit shows up in llvm-config --components.

@pwuertz
Copy link
Author

pwuertz commented Mar 2, 2016

Maybe the problem is in build.py where only a subset of --libs is requested? Maybe a plain llvm-config --libs indeed includes LLVMOProfileJIT as it should (ignoring the extra gtest you reported)?

@pitrou
Copy link
Contributor

pitrou commented Mar 2, 2016

Thanks for suggesting that. It seems that llvm-config doesn't have the bugs it used to have anymore, so perhaps we can use --libs all again. I'll take a look a bit later.

@pitrou pitrou closed this as completed in 5ecb94e Mar 2, 2016
pitrou added a commit that referenced this issue Mar 2, 2016
Fix #121: simplify build process on Unix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants