summaryrefslogtreecommitdiff
path: root/compile.c
diff options
context:
space:
mode:
authorko1 <ko1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2017-11-09 04:27:27 +0000
committerko1 <ko1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2017-11-09 04:27:27 +0000
commit976b6df9596a2077f09f6bed1796a02a229498d1 (patch)
tree67777277e6a6d5063aea2267f8131fdcbc46bd85 /compile.c
parent2b60e342bdc327634e9eade18bad80c854b4ea7c (diff)
fix peephole optimization.
* compile.c (iseq_peephole_optimize): do not need to put `pop` instruction. * test/ruby/test_optimization.rb (test_peephole_optimization_without_trace): This code "def foo; 1.times{|(a), &b| nil && a}; end" fails to compile by stack underflow because of above bug (fixed by this patch). git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60724 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'compile.c')
-rw-r--r--compile.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/compile.c b/compile.c
index 76bc107abe..bcc7dce9fd 100644
--- a/compile.c
+++ b/compile.c
@@ -2324,13 +2324,10 @@ iseq_peephole_optimize(rb_iseq_t *iseq, LINK_ELEMENT *list, const int do_tailcal
* LABEL:
* leave
*/
- INSN *popiobj = new_insn_core(iseq, iobj->line_no,
- BIN(pop), 0, 0);
/* replace */
unref_destination(iobj, 0);
iobj->insn_id = BIN(leave);
iobj->operand_size = 0;
- INSERT_ELEM_NEXT(&iobj->link, &popiobj->link);
goto again;
}
else if ((piobj = (INSN *)get_prev_insn(iobj)) != 0 &&