Skip to content

Commit

Permalink
Thinlto for validator builds. (#3516)
Browse files Browse the repository at this point in the history
# Problem

- Current builds do not use thinlto and we are leaving performance on the table.

# Summary of Changes

- Enable thinlto, the debug-info can't be packed as it gives build error duplicate split compilation unit

# Performance improvements

Results

With thinlto and no debuginformation

[2024-11-23T05:30:31.476026905Z INFO  solana_bench_tps::bench]
    Average max TPS: 74236.48, 0 nodes had 0 TPS
[2024-11-23T05:30:31.476031453Z INFO  solana_bench_tps::bench]
    Highest TPS: 74236.48 sampling period 1s max transactions: 4073007 clients: 1 drop rate: 0.78

Without thinlto and no debuginformation

[2024-11-23T06:24:32.517039914Z INFO  solana_bench_tps::bench] 
    Average max TPS: 66351.46, 0 nodes had 0 TPS
[2024-11-23T06:24:32.517061405Z INFO  solana_bench_tps::bench] 
    Highest TPS: 66351.46 sampling period 1s max transactions: 4302710 clients: 1 drop rate: 0.76
[2024-11-23T06:24:32.517083186Z INFO  solana_bench_tps::bench]  Average TPS: 47228.938
  • Loading branch information
ksolana authored Dec 3, 2024
1 parent 596fd61 commit 5b0e8bc
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@ inherits = "release"
debug = true
split-debuginfo = "packed"

[profile.release]
split-debuginfo = "unpacked"
lto = "thin"

[workspace]
members = [
"account-decoder",
Expand Down

0 comments on commit 5b0e8bc

Please sign in to comment.