Add support for using Tracy with Roc #7706
Merged
+630
−55
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Tracy also helped me find a perf issue with how zig loads files: ziglang/zig#23274 |
lukewilliamboswell
previously approved these changes
Mar 18, 2025
|
Hmm, I don't know why this check just started failing now. I think it is also failing on main, but roc bootstrap has the static version of these deps. Not sure what is going on that is leading to bringing in the homebrew version. |
|
Ok, now I think we are all green with the typo fixed. |
lukewilliamboswell
approved these changes
Mar 18, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Tracy is an awesome profiling tool that helps dive into performance and memory use.
They have an interactive demo at https://tracy.nereid.pl/.
Over the past week, some experiments I have done driven by tracy have lead to reducing the execution time of formatting 100 files of 1000 lines (all the grab bag syntax) by ~40%. Some work was merged with #7704, but more optimization are to follow.
I think that tracy will be a vital tool for creating a version of roc with peak performance. The implementation here is taken from the zig compiler (they also use tracy) and updated for zig version 0.14.0. I have only added a couple of tracing points related to parsing and formatting. I'm sure we will add more stages in the future. As long as tracy is not enable, all of this code is completely cleaned up and becomes a noop.
I tried to also write a general guide on profiling. It is very light, but mentions some tools I have used a lot.
Lastly, this PR updates our build script to: