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

detect LLVM installed #24

Open
ponyatov opened this issue Jun 21, 2020 · 2 comments
Open

detect LLVM installed #24

ponyatov opened this issue Jun 21, 2020 · 2 comments

Comments

@ponyatov
Copy link

make runs local LLVM 10 build, it would be better to use already installed LLVM, at least check and warn that it must be installed manually.

It is important because LLVM build is resource and time-hungry, and that should be done once, and the best way is using the libllvm-dev already provided by host system distribution.

@Araq
Copy link

Araq commented Jun 22, 2020

But then nlvm cannot ensure the quality of the outcome. If nlvm is tested with LLVM 10, it should build with LLVM 10, not with LLVM X just because a "build is resource and time-hungry".

@arnetheduck
Copy link
Owner

I'm happy to take a patch that allows the caller of make to be adventurous with their llvm versions and provide a different one as an option (using an env or make parameter), but generally, nlvm needs a precise version with precise compile options - also, every distro out there is different, so the patch I'd accept would have readme instructions specific to the distro.

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

3 participants