diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-11-22 07:37:13 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-11-22 07:37:13 +0000 |
commit | 792c822ec441a9a061a2c74ddd970005fab33ab6 (patch) | |
tree | f3ae9efa347087acdeb72159b61c3dba6e5b456b /compile.c | |
parent | 53a619de6c1bd36b7c0ab34d273701f78b657d3a (diff) | |
download | ruby-792c822ec441a9a061a2c74ddd970005fab33ab6.tar.gz |
compile.c: optimize useless branches
* compile.c (iseq_peephole_optimize): eliminate always/never
branches after a literal object and when the value is used after
the branch.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@52710 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'compile.c')
-rw-r--r-- | compile.c | 12 |
1 files changed, 11 insertions, 1 deletions
@@ -2053,8 +2053,18 @@ iseq_peephole_optimize(rb_iseq_t *iseq, LINK_ELEMENT *list, const int do_tailcal * => * jump L1 * + * putstring ".." + * dup + * if L1 + * => + * putstring ".." + * jump L1 + * */ int cond; + if (prev_dup && pobj->link.prev->type == ISEQ_ELEMENT_INSN) { + pobj = (INSN *)pobj->link.prev; + } if (pobj->insn_id == BIN(putobject)) { cond = (iobj->insn_id == BIN(branchif) ? OPERAND_AT(pobj, 0) != Qfalse : @@ -2069,7 +2079,7 @@ iseq_peephole_optimize(rb_iseq_t *iseq, LINK_ELEMENT *list, const int do_tailcal cond = iobj->insn_id != BIN(branchif); } else break; - REMOVE_ELEM(&pobj->link); + REMOVE_ELEM(iobj->link.prev); if (cond) { iobj->insn_id = BIN(jump); goto again; |