codegen.c (gen_addsub): use `mrb_int_sub_overflow()`.

c29822f8 · Yukihiro "Matz" Matsumoto · 0455313a · c29822f8
Unverified Commit c29822f8 authored Sep 20, 2021 by Yukihiro "Matz" Matsumoto
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 4 deletions

mrbgems/mruby-compiler/core/codegen.c mrbgems/mruby-compiler/core/codegen.c +5 -4

No files found.
--- a/mrbgems/mruby-compiler/core/codegen.c
+++ b/mrbgems/mruby-compiler/core/codegen.c
@@ -784,11 +784,12 @@ gen_addsub(codegen_scope *s, uint8_t op, uint16_t dst)
      }
      return;
    }
-    if (op == OP_SUB) {
-      if (n == MRB_INT_MIN) goto normal;
-      n = -n;
+    if (op == OP_ADD) {
+      if (mrb_int_add_overflow(n0, n, &n)) goto normal;
+    }
+    else { /* OP_SUB */
+      if (mrb_int_sub_overflow(n0, n, &n)) goto normal;
    }
-    if (mrb_int_add_overflow(n0, n, &n)) goto normal;
    s->pc = addr_pc(s, data0.addr);
    gen_int(s, dst, n);
  }