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

Disable forced reconfiguration of LLVM to speed up build and installation #294

Merged
merged 1 commit into from
May 9, 2018

Conversation

s3rvac
Copy link
Member

@s3rvac s3rvac commented May 8, 2018

This was a relict from our old build system, where we built LLVM as an external project from a subdirectory and needed to detect changes in its source files. Now, we build LLVM as an external project from a GitHub URL, so there is no longer a need to re-configure LLVM after it has been built (its sources will never change).

This speeds up the rebuild and installation of RetDec. For example, on my Linux machine, a rebuild without changes has speeded up from

real    0m1.223s
user    0m2.956s
sys     0m1.163s

to

real    0m0.527s
user    0m1.903s
sys     0m0.716s

On Windows, the difference will be even bigger. This lowering of rebuild times is very handy during development.

…tallation.

This was a relict from our old build system, where we built LLVM as an external
project from a subdirectory and needed to detect changes in its source files.
Now, we build LLVM as an external project from a GitHub URL, so there is no
longer a need to re-configure LLVM after it has been built (its sources will
never change).

This speeds up the rebuild and installation of RetDec. For example, on my Linux
machine, a rebuild without changes has speeded up from

	real    0m1.223s
	user    0m2.956s
	sys     0m1.163s

to

	real    0m0.527s
	user    0m1.903s
	sys     0m0.716s

On Windows, the difference will be even bigger. This lowering of rebuild times
is very handy during development.
@PeterMatula PeterMatula merged commit de20387 into master May 9, 2018
@PeterMatula PeterMatula deleted the llvm-disable-force-reconfiguration branch May 9, 2018 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants