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

[FEAT] Use incremental builds and lto=false for release by default #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

penguincoder
Copy link

This change makes all release profile builds use incremental and lto=false to limit the amount of time it takes to run an individual day. When given --profile, it will add the debug symbols as previously expected.

Background:

As I was working on AoC 2023, the more days that I completed the longer each invocation of cargo aoc would take to run. It eventually got to be nearly 2 minutes in length. I have tweaked the template to always enable incremental builds and to minimize link time optimization to just the current crate in order to reduce build times. For reference, once I rebuilt my current project with the modified cargo-aoc I was able to get my build times down to 5-6 seconds.

This change makes all release profile builds use incremental and
lto=false to limit the amount of time it takes to run an individual day.
When given `--profile`, it will add the debug symbols as previously
expected.
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.

1 participant