-
Notifications
You must be signed in to change notification settings - Fork 14
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
Contribution: Impact reactor logic #136
Comments
FYI I iterated on this a bit and this is my current version
|
Thank you a lot! Suggestion: you can add
In this case, the optimizer is able to inline one more function and the resulting code size is 982 instructions. |
Glad it is helpful! I'm not always sure that the size vs speed trade off is worth it. If the code goes "fast enough" then increasing instruction count makes the code harder for non-mindcode users to read and also makes the schematic string longer. I love how the function inlining lets the compiler bake in things like "this is reactor_1" but it does come at a bit of a cost. |
Hi. I wanted to contribute a working example if you wanted to play with the optimization of someone else's big code.
My current code for this is as follows
Schematic
The text was updated successfully, but these errors were encountered: