admin/mruby-mruby
1
0
Fork 0
mirror of https://github.com/mruby/mruby synced 2026-06-08 16:11:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary OP_JMPNOT for unless statement.

Browse Source
This commit is contained in:
Yukihiro "Matz" Matsumoto
2019-08-17 12:29:04 +09:00
parent e76c5aad36
commit 13eaff4f9f
1 changed files with 22 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
mrbgems/mruby-compiler/core/codegen.c
+22 -12
View File
@@ -1578,18 +1578,17 @@ codegen(codegen_scope *s, node *tree, int val)
} }
codegen(s, tree->car, VAL); codegen(s, tree->car, VAL);
pop(); pop();
pos1 = genjmp2(s, OP_JMPNOT, cursp(), 0, val); if (tree->cdr->car) {
pos1 = genjmp2(s, OP_JMPNOT, cursp(), 0, val);
codegen(s, tree->cdr->car, val); codegen(s, tree->cdr->car, val);
if (elsepart) { if (elsepart) {
if (val) pop(); if (val) pop();
pos2 = genjmp(s, OP_JMP, 0); pos2 = genjmp(s, OP_JMP, 0);
dispatch(s, pos1); dispatch(s, pos1);
codegen(s, elsepart, val); codegen(s, elsepart, val);
dispatch(s, pos2); dispatch(s, pos2);
} }
else { else if (val) {
if (val) {
pop(); pop();
pos2 = genjmp(s, OP_JMP, 0); pos2 = genjmp(s, OP_JMP, 0);
dispatch(s, pos1); dispatch(s, pos1);
@@ -1601,6 +1600,17 @@ codegen(codegen_scope *s, node *tree, int val)
dispatch(s, pos1); dispatch(s, pos1);
} }
} }
else { /* empty then-part */
if (elsepart) {
pos1 = genjmp2(s, OP_JMPIF, cursp(), 0, val);
codegen(s, elsepart, val);
dispatch(s, pos1);
}
else if (val) {
genop_1(s, OP_LOADNIL, cursp());
push();
}
}
} }
break; break;