Skip to content

scriptcomp: disable for loop optimization#142

Merged
niv merged 1 commit intoniv:masterfrom
Daztek:disable-for-loop-opt
Jan 28, 2025
Merged

scriptcomp: disable for loop optimization#142
niv merged 1 commit intoniv:masterfrom
Daztek:disable-for-loop-opt

Conversation

@Daztek
Copy link
Copy Markdown
Contributor

@Daztek Daztek commented Jan 27, 2025

This optimization appears to cause issues with increment/decrement ops, like infinite loops. Disabling it for now.

Open issue: #141
Related PR: #86

Testing

The script in the open issue mentioned above that hit the TMI limit due to an infinite loop now works correctly.

Changelog

Changed

  • scriptcomp: disabled a for loop optimization where the compiler would sometimes generate code that caused an infinite loop.

Licence

  • I am licencing my change under the project's MIT licence, including all changes to GPL-3.0 licenced parts of the codebase.

@niv niv merged commit f407d7a into niv:master Jan 28, 2025
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