nlvm icon indicating copy to clipboard operation
nlvm copied to clipboard

detect LLVM installed

Open ponyatov opened this issue 4 years ago • 2 comments

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.

ponyatov avatar Jun 21 '20 19:06 ponyatov

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".

Araq avatar Jun 22 '20 07:06 Araq

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.

arnetheduck avatar Jun 22 '20 15:06 arnetheduck