perf(release): drop lto=true and bump codegen-units #200

Merged

zaelgohary merged 1 commit from development_perf_release_profile into development

2026-05-03 14:06:00 +00:00

zaelgohary commented

2026-05-03 14:01:37 +00:00

Member

Summary

hero_archipelagos [profile.release] had lto = true and codegen-units = 1, which is the slowest possible config (LTO + no parallel codegen). Since this workspace ships only libraries (no binaries), LTO has nothing to optimize across, but it was still costing ~70s per full workspace build. Daily-sync runs cargo build --release --workspace here once per pass, so the savings compound.

N/A — surfaced while diagnosing daily-sync wallclock.

Changes

lto = true → lto = false
codegen-units = 1 → codegen-units = 16

Test Results

Full clean + workspace build measured locally on herodev:

Before: 2m 49s
After: 1m 37s (43% faster)

## Summary hero_archipelagos `[profile.release]` had `lto = true` and `codegen-units = 1`, which is the slowest possible config (LTO + no parallel codegen). Since this workspace ships only libraries (no binaries), LTO has nothing to optimize across, but it was still costing ~70s per full workspace build. Daily-sync runs `cargo build --release --workspace` here once per pass, so the savings compound. ## Related Issue N/A — surfaced while diagnosing daily-sync wallclock. ## Changes - `lto = true` → `lto = false` - `codegen-units = 1` → `codegen-units = 16` ## Test Results Full clean + workspace build measured locally on herodev: - Before: 2m 49s - After: 1m 37s (43% faster)