wurstscript / WurstScript

Programming language and toolkit to create Warcraft III Maps
https://wurstlang.org
Apache License 2.0
226 stars 28 forks source link

Optimize compilation to Jass #1025

Closed peq closed 2 years ago

peq commented 2 years ago

some minor optimizations:

coveralls commented 2 years ago

Coverage Status

Coverage increased (+0.04%) to 63.498% when pulling 0f4fc3655bde54c4171267dd7083315aed8ae835 on optimize-jass-translation into c3f1e8f038c41bb128b8b05983fa2be5351bb184 on master.

Frotty commented 2 years ago

LGTM, what's missing?

peq commented 2 years ago

There are some differences in the generated code. Seems the optimization behaves differently now, which I did not expect. Need to check which version is right or if the differences are just normal nondeterminism problems.