Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Add a debug and release profile #150

Merged
merged 1 commit into from
Mar 12, 2024
Merged

Add a debug and release profile #150

merged 1 commit into from
Mar 12, 2024

Conversation

MabezDev
Copy link
Member

The rationale behind opt-level = 's' is that it's a good compromise between code size and code performance. On our chips, there is even a relationship between the two, as our caches are quite small so the smaller the code the more we can fit in the cache for better performance. Users can of course update this for their specific needs.

Copy link
Contributor

@bjoernQ bjoernQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - good idea to prevent users from building with opt-level 0

@MabezDev MabezDev merged commit 2f269b1 into main Mar 12, 2024
14 checks passed
@MabezDev MabezDev deleted the add-profiles branch March 12, 2024 16:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants