summaryrefslogtreecommitdiffhomepage
path: root/src/class.c
diff options
context:
space:
mode:
authordearblue <[email protected]>2021-10-29 22:20:20 +0900
committerdearblue <[email protected]>2021-10-29 22:20:20 +0900
commit33792c2a029130bf38d676ae3cf0b51d5d54ffd5 (patch)
treeab5ed91cbb72d3ecf6a9b5fcb5a2b2ee45317606 /src/class.c
parentfca63c6060c3b5825862b713f69451b10ca38aeb (diff)
downloadmruby-33792c2a029130bf38d676ae3cf0b51d5d54ffd5.tar.gz
mruby-33792c2a029130bf38d676ae3cf0b51d5d54ffd5.zip
Improved `Class#new` method
The number of registers used is reduced. Also, previously `R6` and` R7` were used, which exceeded the limit of `new_irep.nregs = 6`. This could cause the VM stack to overrun.
Diffstat (limited to 'src/class.c')
-rw-r--r--src/class.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/class.c b/src/class.c
index 1403ed48e..97a37c54a 100644
--- a/src/class.c
+++ b/src/class.c
@@ -2840,17 +2840,17 @@ static const mrb_code new_iseq[] = {
OP_LOADSELF, 4, // OP_LOADSELF R4
OP_SEND, 4, 0, 0, // OP_SEND R4 :allocate n=0
OP_MOVE, 0, 4, // OP_MOVE R0 R4
- OP_MOVE, 5, 1, // OP_MOVE R5 R1 (*)
- OP_MOVE, 6, 2, // OP_MOVE R6 R2 (**)
- OP_MOVE, 7, 3, // OP_MOVE R7 R3
- OP_SENDB, 4, 1, 255, // OP_SENDB R4 :initialize n=*|nk=*
+ OP_MOVE, 4, 3, // OP_MOVE R4 R3 (&)
+ OP_MOVE, 3, 2, // OP_MOVE R3 R2 (**)
+ OP_MOVE, 2, 1, // OP_MOVE R2 R1 (*)
+ OP_SSENDB, 1, 1, 255, // OP_SSENDB R1 :initialize n=*|nk=*
OP_RETURN, 0 // OP_RETURN R0
};
MRB_PRESYM_DEFINE_VAR_AND_INITER(new_syms, 2, MRB_SYM(allocate), MRB_SYM(initialize))
static const mrb_irep new_irep = {
- 3, 6, 0, MRB_IREP_STATIC,
+ 4, 5, 0, MRB_IREP_STATIC,
new_iseq, NULL, new_syms, NULL, NULL, NULL,
sizeof(new_iseq), 0, 2, 0, 0,
};