fluencelabs/assemblyscript
1
0
Fork 0
mirror of https://github.com/fluencelabs/assemblyscript synced 2025-06-13 23:11:41 +00:00
Code Issues Projects Releases Wiki Activity

Fix continue labels in 'for' loops skipping the incrementor

Browse Source
This commit is contained in:
dcodeIO
2018-05-23 11:32:12 +02:00
parent c897b8405b
commit 829af2285c
36 changed files with 6930 additions and 6625 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/compiler.ts
View File

@ -1601,6 +1601,7 @@ export class Compiler extends DiagnosticEmitter {
flow.breakLabel = breakLabel;
var continueLabel = "continue|" + label;
flow.continueLabel = continueLabel;
var loopLabel = "loop|" + label;
// Compile in correct order
var module = this.module;
@ -1645,13 +1646,16 @@ export class Compiler extends DiagnosticEmitter {
var expr = module.createBlock(breakLabel, [
initExpr,
module.createLoop(continueLabel, module.createBlock(null, [
module.createIf(condExpr, module.createBlock(null, [
bodyExpr,
module.createLoop(loopLabel,
module.createBlock(null, [
module.createBlock(continueLabel, [
module.createBreak(breakLabel, module.createUnary(UnaryOp.EqzI32, condExpr)),
bodyExpr
], NativeType.None),
incrExpr,
module.createBreak(continueLabel)
], NativeType.None))
], NativeType.None))
module.createBreak(loopLabel)
], NativeType.None)
)
], NativeType.None);
// If the loop is guaranteed to run and return, append a hint for Binaryen