Make gen_opt_aref() delegate to gen_opt_send_without_block() (#9)

XrXr · XrXr · commit 52f84419e04e · 2021-10-20T16:31:11.000-04:00
Basically mirroring the interpreter for the general case.
diff --git a/yjit_codegen.c b/yjit_codegen.c
@@ -1125,6 +1125,8 @@ gen_opt_gt(jitstate_t* jit, ctx_t* ctx)
     return gen_fixnum_cmp(jit, ctx, cmovg);
 }
 
+static codegen_status_t gen_opt_send_without_block(jitstate_t *jit, ctx_t *ctx);
+
 static codegen_status_t
 gen_opt_aref(jitstate_t *jit, ctx_t *ctx)
 {
@@ -1266,8 +1268,10 @@ gen_opt_aref(jitstate_t *jit, ctx_t *ctx)
         jit_jump_to_next_insn(jit, ctx);
         return YJIT_END_BLOCK;
     }
-
-    return YJIT_CANT_COMPILE;
+    else {
+        // General case. Call the [] method.
+        return gen_opt_send_without_block(jit, ctx);
+    }
 }
 
 static codegen_status_t

Original file line number	Diff line number	Diff line change
`@@ -1125,6 +1125,8 @@ gen_opt_gt(jitstate_t* jit, ctx_t* ctx)`
`1125`	`1125`	`return gen_fixnum_cmp(jit, ctx, cmovg);`
`1126`	`1126`	`}`
`1127`	`1127`
	`1128`	`+static codegen_status_t gen_opt_send_without_block(jitstate_t jit, ctx_t ctx);`
	`1129`	`+`
`1128`	`1130`	`static codegen_status_t`
`1129`	`1131`	`gen_opt_aref(jitstate_t jit, ctx_t ctx)`
`1130`	`1132`	`{`
`@@ -1266,8 +1268,10 @@ gen_opt_aref(jitstate_t jit, ctx_t ctx)`
`1266`	`1268`	`jit_jump_to_next_insn(jit, ctx);`
`1267`	`1269`	`return YJIT_END_BLOCK;`
`1268`	`1270`	`}`
`1269`		`-`
`1270`		`- return YJIT_CANT_COMPILE;`
	`1271`	`+ else {`
	`1272`	`+ // General case. Call the [] method.`
	`1273`	`+ return gen_opt_send_without_block(jit, ctx);`
	`1274`	`+ }`
`1271`	`1275`	`}`
`1272`	`1276`
`1273`	`1277`	`static codegen_status_t`