-
Notifications
You must be signed in to change notification settings - Fork 953
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
Keep only a single clone of LLVM #14
Comments
Yeah, we are aware. The whole |
Would it be possible to utilize local installations of LLVM on the system? For instance, Arch Linux provides both the latest LLVM release (5.0 package) as a binary package download, and the 3.9 release (3.9 package). For the compile time it would be really wonderful to make use of the LLVM libraries installed on the system. On my laptop, it took 1.5 hours to build retdec (#5 (comment)). Most of that was probably due to building LLVM. |
@mewmew Unfortunately not, because (1) we currently need exactly version 3.9.1 and, most importantly, (2) our version of LLVM contains many customizations that we need for RetDec. Anyhow, even though a cloned repository contains multiple copies of LLVM, only one of them is built. The dependencies should be set correctly. If you believe they are incorrect, please open an issue. |
Yes, LLVM plays a major role in compilation time. No, it is not possible to use local LLVM for the following reasons:
|
just to get a bit clearer overview of what the actual differences are: sandsmark/llvm@3c47bd2 doesn't look impossible to eventually use a vanilla llvm, but it seems like it would require some work to expose the necessary knobs to adjust behavior. |
Options mentioned in duplicate #48:
|
Maybe if you could contact the llvm guys to give options to disable code discarding and optimizations, let it be more configurable, then you guys wouldn't need to hack it and use the vanilla version instead. |
We got rid of all git submodules. See https://github.com/avast-tl/retdec/wiki/Project-Repository-Structure for more details. See #92 how to fix your repo. |
deps/llvm
ANDdeps/fileformat/deps/llvm
ANDdeps/capstone2llvmir/deps/llvm
ANDdeps/fnc-patterns/deps/fileformat/deps/llvm
??Look into conan for sane c++ dependency management...
The text was updated successfully, but these errors were encountered: