Skip to content

[Build] More Build Speed Improvements#4959

Merged
Akkadius merged 1 commit intomasterfrom
akkadius/build-improvements
Jun 30, 2025
Merged

[Build] More Build Speed Improvements#4959
Akkadius merged 1 commit intomasterfrom
akkadius/build-improvements

Conversation

@Akkadius
Copy link
Copy Markdown
Contributor

@Akkadius Akkadius commented Jun 30, 2025

Description

This continues the work of using Clang's Build Analyzer to trace down why the build is slow.

This change further drops the compile time down like a rock. It requires that your cmake build flags are consistent and aren't mixing optimization settings.

Related akk-stack change - https://github.com/Akkadius/akk-stack/commit/c9ce6abc332e44d8b00124f16ce694861d19fdf3

Type of change

  • Build / Compilation Speed Improvement

Testing

16c/32t Intel Linux

Full build

eqemu@ct-eqemu-server:~/code/build$ time ninja -j$(expr $(nproc) - 2)
[540/540] Linking CXX executable bin/zone

real    0m42.737s

Incremental build (edited eqemu_logsys.h)

eqemu@ct-eqemu-server:~/code/build$ time ninja -j$(expr $(nproc) - 2)
[262/262] Linking CXX executable bin/zone

real    0m21.583s

Macbook Air (M4)

Before I started any of this build work, an Macbook Air (M4) used to take roughly 8+ minutes to build

Now it takes

eqemu@ct-eqemu-server:~/code/build$ time ninja -j10
[530/530] Linking CXX executable bin/export_client_files

real	1m33.001s

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

@Akkadius Akkadius force-pushed the akkadius/build-improvements branch from 379c96c to 6c8ebab Compare June 30, 2025 07:41
@Akkadius Akkadius merged commit 07d4845 into master Jun 30, 2025
2 checks passed
@Akkadius Akkadius deleted the akkadius/build-improvements branch June 30, 2025 18:09
@Akkadius Akkadius mentioned this pull request Aug 3, 2025
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

Successfully merging this pull request may close these issues.

2 participants