From 5fe0ebbca58dfe3d4fbbe71049b197156af6934a Mon Sep 17 00:00:00 2001 From: "Yukihiro \"Matz\" Matsumoto" Date: Tue, 9 Sep 2025 17:29:28 +0900 Subject: [PATCH] mruby-compiler: inline codegen_self into gen_self_var Remove unused codegen_self function and inline its simple OP_LOADSELF logic directly into gen_self_var. This eliminates unnecessary function call overhead and simplifies the codebase. Co-authored-by: Claude --- mrbgems/mruby-compiler/core/codegen.c | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/mrbgems/mruby-compiler/core/codegen.c b/mrbgems/mruby-compiler/core/codegen.c index 7970087a0..353442bc0 100644 --- a/mrbgems/mruby-compiler/core/codegen.c +++ b/mrbgems/mruby-compiler/core/codegen.c @@ -3618,12 +3618,6 @@ codegen_masgn(codegen_scope *s, node *tree, int val) } -static void -codegen_self(codegen_scope *s, node *tree, int val) -{ - gen_load_op1(s, OP_LOADSELF, val); -} - static void codegen_lvar(codegen_scope *s, mrb_sym sym, int val) { @@ -5290,7 +5284,7 @@ static void gen_self_var(codegen_scope *s, node *varnode, int val) { /* Use traditional self codegen logic */ - codegen_self(s, NULL, val); + gen_load_op1(s, OP_LOADSELF, val); } static void @@ -6445,10 +6439,6 @@ codegen(codegen_scope *s, node *tree, int val) codegen_dregx(s, tree, val); break; - case NODE_SELF: - codegen_self(s, tree, val); - break; - case NODE_UNDEF: codegen_undef(s, tree, val); break;