codegen.c: `gen_jmpdst` always needs to generate relative address.

b7d75591 · Yukihiro "Matz" Matsumoto · b8863c69 · b7d75591
Unverified Commit b7d75591 authored Jul 15, 2021 by Yukihiro "Matz" Matsumoto
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 8 deletions

mrbgems/mruby-compiler/core/codegen.c mrbgems/mruby-compiler/core/codegen.c +6 -8

No files found.
--- a/mrbgems/mruby-compiler/core/codegen.c
+++ b/mrbgems/mruby-compiler/core/codegen.c
@@ -398,17 +398,15 @@ gen_jmpdst(codegen_scope *s, uint32_t pc)
 {

  if (pc == JMPLINK_START) {
-    gen_S(s, 0);
+    pc = 0;
  }
-  else {
-    uint32_t pos2 = s->pc+2;
-    int32_t off = pc - pos2;
+  uint32_t pos2 = s->pc+2;
+  int32_t off = pc - pos2;

-    if (off > INT16_MAX || INT16_MIN > off) {
-      codegen_error(s, "too big jump offset");
-    }
-    gen_S(s, (uint16_t)off);
+  if (off > INT16_MAX || INT16_MIN > off) {
+    codegen_error(s, "too big jump offset");
  }
+  gen_S(s, (uint16_t)off);
 }

 static uint32_t