Merge pull request #295 from TurboWarp/optimize-loops

GarboMuffin · web-flow · commit 51aacba14c9b · 2025-10-05T08:54:44.000-05:00
Optimize loops with provably-non-fraction iteration counts
diff --git a/src/compiler/jsgen.js b/src/compiler/jsgen.js
@@ -610,7 +610,11 @@ class JSGenerator {
             break;
         case StackOpcode.CONTROL_REPEAT: {
             const i = this.localVariables.next();
-            this.source += `for (var ${i} = ${this.descendInput(node.times)}; ${i} >= 0.5; ${i}--) {\n`;
+            if (node.times.isAlwaysType(InputType.NUMBER_INT | InputType.NUMBER_INF)) {
+                this.source += `for (var ${i} = ${this.descendInput(node.times)}; ${i} > 0; ${i}--) {\n`;
+            } else {
+                this.source += `for (var ${i} = ${this.descendInput(node.times)}; ${i} >= 0.5; ${i}--) {\n`;
+            }
             this.descendStack(node.do, new Frame(true));
             this.yieldLoop();
             this.source += `}\n`;
diff --git a/test/snapshot/__snapshots__/tw-analyze-yields-due-to-direct-recursion.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-analyze-yields-due-to-direct-recursion.sb3.tw-snapshot
@@ -31,7 +31,7 @@ return "";
 (function factoryXYZ(thread) { const target = thread.target; const runtime = target.runtime; const stage = runtime.getTargetForStage();
 const b0 = stage.variables["`jEk@4|i[#Fk?(8x)AV.-my variable"];
 return function* genXYZ () {
-for (var a0 = 1; a0 >= 0.5; a0--) {
+for (var a0 = 1; a0 > 0; a0--) {
 yield;
 }
 b0.value = "random";
diff --git a/test/snapshot/__snapshots__/tw-comparison-matrix-runtime.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-comparison-matrix-runtime.sb3.tw-snapshot
@@ -23,10 +23,10 @@ return function* genXYZ_run_test () {
 thread.procedures["Wsetup values"]();
 b0.value = 0;
 b1.value = 0;
-for (var a0 = b2.value.length; a0 >= 0.5; a0--) {
+for (var a0 = b2.value.length; a0 > 0; a0--) {
 b1.value = ((+b1.value || 0) + 1);
 b3.value = 0;
-for (var a1 = b2.value.length; a1 >= 0.5; a1--) {
+for (var a1 = b2.value.length; a1 > 0; a1--) {
 b3.value = ((+b3.value || 0) + 1);
 b0.value = ((+b0.value || 0) + 1);
 if (!compareEqual(compareGreaterThan(listGet(b2.value, b1.value), (b2.value[(b3.value | 0) - 1] ?? "")), (b4.value[(b0.value | 0) - 1] ?? ""))) {
diff --git a/test/snapshot/__snapshots__/tw-counter.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-counter.sb3.tw-snapshot
@@ -32,7 +32,7 @@ runtime.ext_scratch3_control._counter = 0;
 if ((runtime.ext_scratch3_control._counter === 0)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass clear = 0",}, b0, false, false, "w", null);
 }
-for (var a0 = 10; a0 >= 0.5; a0--) {
+for (var a0 = 10; a0 > 0; a0--) {
 runtime.ext_scratch3_control._counter++;
 yield;
 }
diff --git a/test/snapshot/__snapshots__/tw-exit-state-accounts-for-yields-in-procedures.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-exit-state-accounts-for-yields-in-procedures.sb3.tw-snapshot
@@ -20,7 +20,7 @@ retire(); return;
 // Sprite1 Zsomething that yields
 (function factoryXYZ(thread) { const target = thread.target; const runtime = target.runtime; const stage = runtime.getTargetForStage();
 return function* genXYZ_something_that_yield () {
-for (var a0 = 2; a0 >= 0.5; a0--) {
+for (var a0 = 2; a0 > 0; a0--) {
 yield;
 }
 return "";
@@ -30,7 +30,7 @@ return "";
 (function factoryXYZ(thread) { const target = thread.target; const runtime = target.runtime; const stage = runtime.getTargetForStage();
 const b0 = stage.variables["`jEk@4|i[#Fk?(8x)AV.-my variable"];
 return function* genXYZ () {
-for (var a0 = 1; a0 >= 0.5; a0--) {
+for (var a0 = 1; a0 > 0; a0--) {
 yield;
 }
 b0.value = "random";
diff --git a/test/snapshot/__snapshots__/tw-forkphorus-515-wait-zero-seconds-in-warp-mode.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-forkphorus-515-wait-zero-seconds-in-warp-mode.sb3.tw-snapshot
@@ -26,7 +26,7 @@ retire(); return;
 const b0 = runtime.getOpcodeFunction("music_restForBeats");
 const b1 = runtime.getOpcodeFunction("music_playDrumForBeats");
 return function* genXYZ_no_refresh () {
-for (var a0 = 30; a0 >= 0.5; a0--) {
+for (var a0 = 30; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
@@ -53,7 +53,7 @@ return "";
 const b0 = runtime.getOpcodeFunction("music_restForBeats");
 const b1 = runtime.getOpcodeFunction("music_playDrumForBeats");
 return function* genXYZ_has_refresh () {
-for (var a0 = 30; a0 >= 0.5; a0--) {
+for (var a0 = 30; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
diff --git a/test/snapshot/__snapshots__/tw-gh-249-quicksort.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-gh-249-quicksort.sb3.tw-snapshot
@@ -18,7 +18,7 @@ const b0 = stage.variables["EF^k=u-t@S)w60;RP?dZ-list-list"];
 const b1 = stage.variables["rQq!wAuU2T9DV9,S[/s."];
 return function funXYZ_run () {
 b0.value = [];
-for (var a0 = 100; a0 >= 0.5; a0--) {
+for (var a0 = 100; a0 > 0; a0--) {
 b0.value.push((b1.value[(b0.value.length + 1) - 1] ?? ""));
 b0._monitorUpToDate = false;
 }
@@ -36,7 +36,7 @@ const b4 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_validate () {
 b0.value = 1;
 b1.value = 1;
-for (var a0 = b2.value.length; a0 >= 0.5; a0--) {
+for (var a0 = b2.value.length; a0 > 0; a0--) {
 if (!compareEqual((b2.value[(b1.value | 0) - 1] ?? ""), (b3.value[(b1.value | 0) - 1] ?? ""))) {
 b0.value = 0;
 yield* executeInCompatibilityLayer({"MESSAGE":("fail mismatch at index " + ("" + b1.value)),}, b4, true, false, ",", null);
diff --git a/test/snapshot/__snapshots__/tw-preciseProjectTimer-drift-453118719.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-preciseProjectTimer-drift-453118719.sb3.tw-snapshot
@@ -9,7 +9,7 @@ const b2 = stage.variables["l^q!%fq]Bv;72dlGf}^Z"];
 return function* genXYZ () {
 b0.value = 0;
 yield* executeInCompatibilityLayer({"MESSAGE":"plan 0",}, b1, false, false, "}fY]VN(X|v/[G`P0?@s2", null);
-for (var a0 = 30; a0 >= 0.5; a0--) {
+for (var a0 = 30; a0 > 0; a0--) {
 b2.value = runtime.ioDevices.clock.projectTimer();
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
diff --git a/test/snapshot/__snapshots__/tw-procedure-return-stops-scripts.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-procedure-return-stops-scripts.sb3.tw-snapshot
@@ -25,7 +25,7 @@ const b0 = stage.variables["PsAI*C{QHI3*4?O8p#TM"];
 const b1 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_return_stops_the_scr () {
 b0.value = 0;
-for (var a0 = 100; a0 >= 0.5; a0--) {
+for (var a0 = 100; a0 > 0; a0--) {
 b0.value = (b0.value + 1);
 if ((b0.value === 25)) {
 return "stopped!";
diff --git a/test/snapshot/__snapshots__/tw-procedure-return-warp.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-procedure-return-warp.sb3.tw-snapshot
@@ -38,7 +38,7 @@ const b0 = stage.variables["Go=PJS7BFXYo_qi2S:kQ"];
 const b1 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_non_warp () {
 b0.value = 0;
-for (var a0 = 5; a0 >= 0.5; a0--) {
+for (var a0 = 5; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
@@ -56,7 +56,7 @@ if ((("" + (yield* thread.procedures["Wverify runs warp %s"]((yield* thread.proc
 yield* executeInCompatibilityLayer({"MESSAGE":"pass non warp and warp mix",}, b1, false, false, "S", null);
 }
 b0.value = 0;
-for (var a2 = 5; a2 >= 0.5; a2--) {
+for (var a2 = 5; a2 > 0; a2--) {
 thread.timer = timer();
 var a3 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
@@ -79,7 +79,7 @@ const b0 = stage.variables["Go=PJS7BFXYo_qi2S:kQ"];
 const b1 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_verify_runs_warp_ (p0) {
 b0.value = 0;
-for (var a0 = 5; a0 >= 0.5; a0--) {
+for (var a0 = 5; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
@@ -101,7 +101,7 @@ const b0 = stage.variables["Go=PJS7BFXYo_qi2S:kQ"];
 const b1 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_verify_runs_non_warp (p0) {
 b0.value = 0;
-for (var a0 = 5; a0 >= 0.5; a0--) {
+for (var a0 = 5; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
diff --git a/test/snapshot/__snapshots__/tw-promise-loop-double-yield-kouzeru.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-promise-loop-double-yield-kouzeru.sb3.tw-snapshot
@@ -29,7 +29,7 @@ const b2 = stage.variables["JbF5exWEi*m?-UEmkASS"];
 return function* genXYZ () {
 b0.value = 0;
 yield* executeInCompatibilityLayer({"MESSAGE":"plan 15",}, b1, false, false, "0/2-)Gp^^=haQ1OMD.xb", null);
-for (var a0 = 15; a0 >= 0.5; a0--) {
+for (var a0 = 15; a0 > 0; a0--) {
 if (((+b2.value || 0) === 200)) {
 b2.value = 50;
 } else {
diff --git a/test/snapshot/__snapshots__/tw-self-restarting-script-keeps-running-until-yield.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-self-restarting-script-keeps-running-until-yield.sb3.tw-snapshot
@@ -24,7 +24,7 @@ if (compareEqual(b0.value, 0)) {
 b0.value = 1;
 startHats("event_whenbroadcastreceived", { BROADCAST_OPTION: "message1" });
 b1.value = 1;
-for (var a0 = 1; a0 >= 0.5; a0--) {
+for (var a0 = 1; a0 > 0; a0--) {
 yield;
 }
 b1.value = 2;
diff --git a/test/snapshot/__snapshots__/tw-unsafe-equals.sb3.tw-snapshot b/test/snapshot/__snapshots__/tw-unsafe-equals.sb3.tw-snapshot
@@ -24,7 +24,7 @@ yield* executeInCompatibilityLayer({"MESSAGE":"pass 4",}, b0, false, false, "8]2
 if (((+b1.value || 0) === 10)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass 5",}, b0, false, false, "ZU^{OfKTg|+Au$$q0[]u", null);
 }
-for (var a0 = 1; a0 >= 0.5; a0--) {
+for (var a0 = 1; a0 > 0; a0--) {
 if (((+b1.value || 0) === 10)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass 6",}, b0, false, false, "HB+_IN}6=K[*ksxKXH0`", null);
 }
@@ -47,7 +47,7 @@ yield* executeInCompatibilityLayer({"MESSAGE":"pass 10",}, b0, false, false, "B`
 if (((+b2.value || 0) === 10)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass 11",}, b0, false, false, "TJ:#TkYBys*!RYiKLXb)", null);
 }
-for (var a1 = 1; a1 >= 0.5; a1--) {
+for (var a1 = 1; a1 > 0; a1--) {
 if (((+b2.value || 0) === 10)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass 12",}, b0, false, false, ",Z,~10Qo~j;(+VL+I3q:", null);
 }
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-analyze-yields-due-to-direct-recursion.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-analyze-yields-due-to-direct-recursion.sb3.tw-snapshot
@@ -31,7 +31,7 @@ return "";
 (function factoryXYZ(thread) { const target = thread.target; const runtime = target.runtime; const stage = runtime.getTargetForStage();
 const b0 = stage.variables["`jEk@4|i[#Fk?(8x)AV.-my variable"];
 return function* genXYZ () {
-for (var a0 = 1; a0 >= 0.5; a0--) {
+for (var a0 = 1; a0 > 0; a0--) {
 yield;
 }
 b0.value = "random";
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-comparison-matrix-runtime.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-comparison-matrix-runtime.sb3.tw-snapshot
@@ -23,10 +23,10 @@ return function* genXYZ_run_test () {
 thread.procedures["Wsetup values"]();
 b0.value = 0;
 b1.value = 0;
-for (var a0 = b2.value.length; a0 >= 0.5; a0--) {
+for (var a0 = b2.value.length; a0 > 0; a0--) {
 b1.value = ((+b1.value || 0) + 1);
 b3.value = 0;
-for (var a1 = b2.value.length; a1 >= 0.5; a1--) {
+for (var a1 = b2.value.length; a1 > 0; a1--) {
 b3.value = ((+b3.value || 0) + 1);
 b0.value = ((+b0.value || 0) + 1);
 if (!compareEqual(compareGreaterThan(listGet(b2.value, b1.value), (b2.value[(b3.value | 0) - 1] ?? "")), (b4.value[(b0.value | 0) - 1] ?? ""))) {
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-counter.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-counter.sb3.tw-snapshot
@@ -32,7 +32,7 @@ runtime.ext_scratch3_control._counter = 0;
 if ((runtime.ext_scratch3_control._counter === 0)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass clear = 0",}, b0, false, false, "w", null);
 }
-for (var a0 = 10; a0 >= 0.5; a0--) {
+for (var a0 = 10; a0 > 0; a0--) {
 runtime.ext_scratch3_control._counter++;
 yield;
 }
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-exit-state-accounts-for-yields-in-procedures.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-exit-state-accounts-for-yields-in-procedures.sb3.tw-snapshot
@@ -20,7 +20,7 @@ retire(); return;
 // Sprite1 Zsomething that yields
 (function factoryXYZ(thread) { const target = thread.target; const runtime = target.runtime; const stage = runtime.getTargetForStage();
 return function* genXYZ_something_that_yield () {
-for (var a0 = 2; a0 >= 0.5; a0--) {
+for (var a0 = 2; a0 > 0; a0--) {
 yield;
 }
 return "";
@@ -30,7 +30,7 @@ return "";
 (function factoryXYZ(thread) { const target = thread.target; const runtime = target.runtime; const stage = runtime.getTargetForStage();
 const b0 = stage.variables["`jEk@4|i[#Fk?(8x)AV.-my variable"];
 return function* genXYZ () {
-for (var a0 = 1; a0 >= 0.5; a0--) {
+for (var a0 = 1; a0 > 0; a0--) {
 yield;
 }
 b0.value = "random";
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-forkphorus-515-wait-zero-seconds-in-warp-mode.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-forkphorus-515-wait-zero-seconds-in-warp-mode.sb3.tw-snapshot
@@ -26,7 +26,7 @@ retire(); return;
 const b0 = runtime.getOpcodeFunction("music_restForBeats");
 const b1 = runtime.getOpcodeFunction("music_playDrumForBeats");
 return function* genXYZ_no_refresh () {
-for (var a0 = 30; a0 >= 0.5; a0--) {
+for (var a0 = 30; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
@@ -54,7 +54,7 @@ return "";
 const b0 = runtime.getOpcodeFunction("music_restForBeats");
 const b1 = runtime.getOpcodeFunction("music_playDrumForBeats");
 return function* genXYZ_has_refresh () {
-for (var a0 = 30; a0 >= 0.5; a0--) {
+for (var a0 = 30; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-gh-249-quicksort.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-gh-249-quicksort.sb3.tw-snapshot
@@ -18,7 +18,7 @@ const b0 = stage.variables["EF^k=u-t@S)w60;RP?dZ-list-list"];
 const b1 = stage.variables["rQq!wAuU2T9DV9,S[/s."];
 return function* genXYZ_run () {
 b0.value = [];
-for (var a0 = 100; a0 >= 0.5; a0--) {
+for (var a0 = 100; a0 > 0; a0--) {
 b0.value.push((b1.value[(b0.value.length + 1) - 1] ?? ""));
 b0._monitorUpToDate = false;
 if (isStuck()) yield;
@@ -37,7 +37,7 @@ const b4 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_validate () {
 b0.value = 1;
 b1.value = 1;
-for (var a0 = b2.value.length; a0 >= 0.5; a0--) {
+for (var a0 = b2.value.length; a0 > 0; a0--) {
 if (!compareEqual(listGet(b2.value, b1.value), listGet(b3.value, b1.value))) {
 b0.value = 0;
 yield* executeInCompatibilityLayer({"MESSAGE":("fail mismatch at index " + ("" + b1.value)),}, b4, true, false, ",", null);
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-preciseProjectTimer-drift-453118719.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-preciseProjectTimer-drift-453118719.sb3.tw-snapshot
@@ -9,7 +9,7 @@ const b2 = stage.variables["l^q!%fq]Bv;72dlGf}^Z"];
 return function* genXYZ () {
 b0.value = 0;
 yield* executeInCompatibilityLayer({"MESSAGE":"plan 0",}, b1, false, false, "}fY]VN(X|v/[G`P0?@s2", null);
-for (var a0 = 30; a0 >= 0.5; a0--) {
+for (var a0 = 30; a0 > 0; a0--) {
 b2.value = runtime.ioDevices.clock.projectTimer();
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-procedure-return-stops-scripts.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-procedure-return-stops-scripts.sb3.tw-snapshot
@@ -25,7 +25,7 @@ const b0 = stage.variables["PsAI*C{QHI3*4?O8p#TM"];
 const b1 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_return_stops_the_scr () {
 b0.value = 0;
-for (var a0 = 100; a0 >= 0.5; a0--) {
+for (var a0 = 100; a0 > 0; a0--) {
 b0.value = ((+b0.value || 0) + 1);
 if ((b0.value === 25)) {
 return "stopped!";
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-procedure-return-warp.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-procedure-return-warp.sb3.tw-snapshot
@@ -38,7 +38,7 @@ const b0 = stage.variables["Go=PJS7BFXYo_qi2S:kQ"];
 const b1 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_non_warp () {
 b0.value = 0;
-for (var a0 = 5; a0 >= 0.5; a0--) {
+for (var a0 = 5; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
@@ -56,7 +56,7 @@ if ((("" + (yield* thread.procedures["Wverify runs warp %s"]((yield* thread.proc
 yield* executeInCompatibilityLayer({"MESSAGE":"pass non warp and warp mix",}, b1, false, false, "S", null);
 }
 b0.value = 0;
-for (var a2 = 5; a2 >= 0.5; a2--) {
+for (var a2 = 5; a2 > 0; a2--) {
 thread.timer = timer();
 var a3 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
@@ -79,7 +79,7 @@ const b0 = stage.variables["Go=PJS7BFXYo_qi2S:kQ"];
 const b1 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_verify_runs_warp_ (p0) {
 b0.value = 0;
-for (var a0 = 5; a0 >= 0.5; a0--) {
+for (var a0 = 5; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
@@ -102,7 +102,7 @@ const b0 = stage.variables["Go=PJS7BFXYo_qi2S:kQ"];
 const b1 = runtime.getOpcodeFunction("looks_say");
 return function* genXYZ_verify_runs_non_warp (p0) {
 b0.value = 0;
-for (var a0 = 5; a0 >= 0.5; a0--) {
+for (var a0 = 5; a0 > 0; a0--) {
 thread.timer = timer();
 var a1 = Math.max(0, 1000 * 0);
 runtime.requestRedraw();
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-promise-loop-double-yield-kouzeru.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-promise-loop-double-yield-kouzeru.sb3.tw-snapshot
@@ -29,7 +29,7 @@ const b2 = stage.variables["JbF5exWEi*m?-UEmkASS"];
 return function* genXYZ () {
 b0.value = 0;
 yield* executeInCompatibilityLayer({"MESSAGE":"plan 15",}, b1, false, false, "0/2-)Gp^^=haQ1OMD.xb", null);
-for (var a0 = 15; a0 >= 0.5; a0--) {
+for (var a0 = 15; a0 > 0; a0--) {
 if (((+b2.value || 0) === 200)) {
 b2.value = 50;
 } else {
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-self-restarting-script-keeps-running-until-yield.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-self-restarting-script-keeps-running-until-yield.sb3.tw-snapshot
@@ -24,7 +24,7 @@ if (compareEqual(b0.value, 0)) {
 b0.value = 1;
 startHats("event_whenbroadcastreceived", { BROADCAST_OPTION: "message1" });
 b1.value = 1;
-for (var a0 = 1; a0 >= 0.5; a0--) {
+for (var a0 = 1; a0 > 0; a0--) {
 yield;
 }
 b1.value = 2;
diff --git a/test/snapshot/__snapshots__/warp-timer/tw-unsafe-equals.sb3.tw-snapshot b/test/snapshot/__snapshots__/warp-timer/tw-unsafe-equals.sb3.tw-snapshot
@@ -24,7 +24,7 @@ yield* executeInCompatibilityLayer({"MESSAGE":"pass 4",}, b0, false, false, "8]2
 if (((+b1.value || 0) === 10)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass 5",}, b0, false, false, "ZU^{OfKTg|+Au$$q0[]u", null);
 }
-for (var a0 = 1; a0 >= 0.5; a0--) {
+for (var a0 = 1; a0 > 0; a0--) {
 if (((+b1.value || 0) === 10)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass 6",}, b0, false, false, "HB+_IN}6=K[*ksxKXH0`", null);
 }
@@ -47,7 +47,7 @@ yield* executeInCompatibilityLayer({"MESSAGE":"pass 10",}, b0, false, false, "B`
 if (((+b2.value || 0) === 10)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass 11",}, b0, false, false, "TJ:#TkYBys*!RYiKLXb)", null);
 }
-for (var a1 = 1; a1 >= 0.5; a1--) {
+for (var a1 = 1; a1 > 0; a1--) {
 if (((+b2.value || 0) === 10)) {
 yield* executeInCompatibilityLayer({"MESSAGE":"pass 12",}, b0, false, false, ",Z,~10Qo~j;(+VL+I3q:", null);
 }

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ return "";`
`31`	`31`	`(function factoryXYZ(thread) { const target = thread.target; const runtime = target.runtime; const stage = runtime.getTargetForStage();`
`32`	`32`	const b0 = stage.variables["`jEk@4\|i[#Fk?(8x)AV.-my variable"];
`33`	`33`	`return function* genXYZ () {`
`34`		`-for (var a0 = 1; a0 >= 0.5; a0--) {`
	`34`	`+for (var a0 = 1; a0 > 0; a0--) {`
`35`	`35`	`yield;`
`36`	`36`	`}`
`37`	`37`	`b0.value = "random";`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ runtime.ext_scratch3_control._counter = 0;`
`32`	`32`	`if ((runtime.ext_scratch3_control._counter === 0)) {`
`33`	`33`	`yield* executeInCompatibilityLayer({"MESSAGE":"pass clear = 0",}, b0, false, false, "w", null);`
`34`	`34`	`}`
`35`		`-for (var a0 = 10; a0 >= 0.5; a0--) {`
	`35`	`+for (var a0 = 10; a0 > 0; a0--) {`
`36`	`36`	`runtime.ext_scratch3_control._counter++;`
`37`	`37`	`yield;`
`38`	`38`	`}`