From 7d6b8024b56f59289baba5bc94a4ab5bf82f8298 Mon Sep 17 00:00:00 2001 From: "Yukihiro \"Matz\" Matsumoto" Date: Wed, 1 Dec 2021 14:28:27 +0900 Subject: codegen.c: fix a bug in `OP_ADDI` and `OP_SUBI` regarding negative values. --- mrbgems/mruby-compiler/core/codegen.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) (limited to 'mrbgems') diff --git a/mrbgems/mruby-compiler/core/codegen.c b/mrbgems/mruby-compiler/core/codegen.c index fa755e1e7..a1cde4d7d 100644 --- a/mrbgems/mruby-compiler/core/codegen.c +++ b/mrbgems/mruby-compiler/core/codegen.c @@ -779,13 +779,16 @@ gen_addsub(codegen_scope *s, uint8_t op, uint16_t dst) mrb_int n0; if (addr_pc(s, data.addr) == s->lastlabel || !get_int_operand(s, &data0, &n0)) { /* OP_ADDI/OP_SUBI takes upto 16bits */ - if (n > INT16_MAX) goto normal; + if (n > INT16_MAX || n < INT16_MIN) goto normal; rewind_pc(s); - if (op == OP_ADD) { - genop_2(s, OP_ADDI, dst, (uint16_t)n); - } - else { - genop_2(s, OP_SUBI, dst, (uint16_t)n); + if (n > 0) { + if (op == OP_ADD) genop_2(s, OP_ADDI, dst, (uint16_t)n); + else genop_2(s, OP_SUBI, dst, (uint16_t)n); + } + else { /* n < 0 */ + n = -n; + if (op == OP_ADD) genop_2(s, OP_SUBI, dst, (uint16_t)n); + else genop_2(s, OP_ADDI, dst, (uint16_t)n); } return; } -- cgit v1.2.3