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

Run compiler specs in release mode #13122

Merged
merged 2 commits into from
Feb 28, 2023
Merged

Conversation

straight-shoota
Copy link
Member

Building the compiler (or compiler specs in this case) in release mode has a hefty upfront cost, but it improves execution speed of the compiler a lot. The extra overhead is not worth it if you only use the compiler a couple of times, but when you use it a lot, it makes sense to build in release mode.

The compiler specs are pretty huge and thus use the compiler a lot. So I figured, why not try build compiler_spec in release mode to speed up the execution? Compiler specs are a huge portion of CI job runtime, so maybe we can speed things up a bit.

So I did that in this branch and compared the runtimes with previous runs on master.
The duration of spec runs varies quite a bit, actually. So it must be taken with a grain of salt. I did only one CI run in release mode yet.
But there's a clear tendency: Each individual CI job which runs compiler_spec finished more quickly in release mode than in previous runs. The speed up is typically between 3-10 minutes.

As an example, this is the comparison of the x86_64-gnu-test jobs in Linux CI:

This branch #13079
35:59 45:13
42:57 49:32
35:26 46:01
43:40 55:58
45:27 48:42
35:45 45:16
34:49 46:18
40:05 45:27

The total sum of durations dropeed from 6:22:27 to 5:14:08.

So running specs in release mode seems to bring quite a noticable performance improvement.

Makefile Show resolved Hide resolved
@straight-shoota straight-shoota added this to the 1.8.0 milestone Feb 27, 2023
@straight-shoota straight-shoota merged commit d747d21 into master Feb 28, 2023
@straight-shoota straight-shoota deleted the test/compiler-specs-release branch February 28, 2023 20:34
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.

2 participants