Skip to content

Conversation

@simonmarty
Copy link
Contributor

@simonmarty simonmarty commented Nov 18, 2024

Issue #, if available: See #9

https://doc.rust-lang.org/cargo/reference/profiles.html

Description of changes:

Experimenting with release profile parameters

  • lto = "thin to use link time optimization across the dependency tree. This increases the binary size by 0.5MB on stable-x86_64-apple-darwin. At the moment I'm not seeing a significant performance disparity between this being set to "thin" or "off". This might differ between platforms.
  • codegen-units = 1, significantly increases compilation time, is supposed to help with optimization, does appear to reduce binary size by a little over 1MB.
  • strip = true Removes symbols and debug info, reduces binary size (13MB -> 9MB).

Before merging:

Run a load test and verify this leads to a consistent performance improvement on

  • stable-x86_64-unknown-linux-gnu
  • stable-x86_64-unknown-linux-musl
  • stable-aarch64-unknown-linux-gnu
  • stable-aarch64-unknown-linux-musl
  • stable-x86_64-pc-windows-msvc

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov-commenter
Copy link

codecov-commenter commented Jan 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.72%. Comparing base (8f90c3b) to head (9b85572).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   91.72%   91.72%           
=======================================
  Files          14       14           
  Lines        2418     2418           
  Branches     2418     2418           
=======================================
  Hits         2218     2218           
  Misses        150      150           
  Partials       50       50           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@simonmarty simonmarty force-pushed the main branch 2 times, most recently from de875e6 to cde666a Compare October 25, 2025 00:58
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