Skip to content

Dispatch type-specific opt insn from send (take 2) #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
k0kubun wants to merge 16 commits into from
Closed

Dispatch type-specific opt insn from send (take 2) #10

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
91ef66e
insns.def: drop opt_plus insn
k0kubun Sep 18, 2018
cd68b7e
vm_insnhelper.c: split vm_opt_plus
k0kubun Sep 19, 2018
ed7bff2
_mjit_compile_send.erb: inline opt cc->call
k0kubun Sep 21, 2018
b105cd1
vm_insnhelper.h: add RB_DEFINE_FASTPATH_INLINE
k0kubun Sep 22, 2018
fffaf59
vm_insnhelper.h: always inline
k0kubun Sep 22, 2018
b888472
vm_insnhelper.c: split vm_opt_minus
k0kubun Sep 22, 2018
7695a78
vm_insnhelper.c: split vm_opt_mult
k0kubun Sep 23, 2018
9179aab
vm_insnhelper.c: split vm_opt_div
k0kubun Sep 23, 2018
2423fbf
vm_insnhelper.c: split vm_opt_mod
k0kubun Sep 23, 2018
25d29cd
vm_insnhelper.c: split lt, le, gt, ge
k0kubun Sep 23, 2018
f7dcf04
vm_insnhelper.c: fix opt mod bug
k0kubun Sep 23, 2018
1724611
vm_insnhelper.c: split vm_opt_aref
k0kubun Sep 23, 2018
1e4f897
vm_insnhelper.c: split length, size, empty?
k0kubun Sep 23, 2018
dc3ea08
vm_insnhelper.c: guard against __send__
k0kubun Sep 23, 2018
460ddfc
vm_insnhelper.c: split vm_opt_succ
k0kubun Sep 23, 2018
c926f89
inline argc == 0 case (not tested yet)
k0kubun Sep 25, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
insns.def: drop opt_plus insn
  • Loading branch information
@k0kubun
k0kubun committed Sep 19, 2018
commit 91ef66e3d7feb060616d5630ead78eebe4c7640b
1 change: 0 additions & 1 deletion compile.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3232,7 +3232,6 @@ iseq_specialized_instruction(rb_iseq_t *iseq, INSN *iobj)
break;
case 1:
switch (ci->mid) {
case idPLUS: SP_INSN(plus); return COMPILE_OK;
case idMINUS: SP_INSN(minus); return COMPILE_OK;
case idMULT: SP_INSN(mult); return COMPILE_OK;
case idDIV: SP_INSN(div); return COMPILE_OK;
Expand Down
17 changes: 0 additions & 17 deletions insns.def
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1040,23 +1040,6 @@ opt_case_dispatch

/** simple functions */

/* optimized X+Y. */
DEFINE_INSN
opt_plus
(CALL_INFO ci, CALL_CACHE cc)
(VALUE recv, VALUE obj)
(VALUE val)
/* Array + anything can be handled inside of opt_plus, and that
* anything is converted into array using #to_ary. */
// attr bool leaf = false; /* has rb_to_array_type() */
{
val = vm_opt_plus(recv, obj);

if (val == Qundef) {
CALL_SIMPLE_METHOD();
}
}

/* optimized X-Y. */
DEFINE_INSN
opt_minus
Expand Down
33 changes: 0 additions & 33 deletions vm_insnhelper.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3423,39 +3423,6 @@ vm_stack_consistency_error(const rb_execution_context_t *ec,
#endif
}

static VALUE
vm_opt_plus(VALUE recv, VALUE obj)
{
if (FIXNUM_2_P(recv, obj) &&
BASIC_OP_UNREDEFINED_P(BOP_PLUS, INTEGER_REDEFINED_OP_FLAG)) {
return rb_fix_plus_fix(recv, obj);
}
else if (FLONUM_2_P(recv, obj) &&
BASIC_OP_UNREDEFINED_P(BOP_PLUS, FLOAT_REDEFINED_OP_FLAG)) {
return DBL2NUM(RFLOAT_VALUE(recv) + RFLOAT_VALUE(obj));
}
else if (SPECIAL_CONST_P(recv) || SPECIAL_CONST_P(obj)) {
return Qundef;
}
else if (RBASIC_CLASS(recv) == rb_cFloat &&
RBASIC_CLASS(obj) == rb_cFloat &&
BASIC_OP_UNREDEFINED_P(BOP_PLUS, FLOAT_REDEFINED_OP_FLAG)) {
return DBL2NUM(RFLOAT_VALUE(recv) + RFLOAT_VALUE(obj));
}
else if (RBASIC_CLASS(recv) == rb_cString &&
RBASIC_CLASS(obj) == rb_cString &&
BASIC_OP_UNREDEFINED_P(BOP_PLUS, STRING_REDEFINED_OP_FLAG)) {
return rb_str_plus(recv, obj);
}
else if (RBASIC_CLASS(recv) == rb_cArray &&
BASIC_OP_UNREDEFINED_P(BOP_PLUS, ARRAY_REDEFINED_OP_FLAG)) {
return rb_ary_plus(recv, obj);
}
else {
return Qundef;
}
}

static VALUE
vm_opt_minus(VALUE recv, VALUE obj)
{
Expand Down