summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2016-10-29 01:09:40 +0000
committernobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2016-10-29 01:09:40 +0000
commitaf1b9db40b31f5ef186799345931e1de59843892 (patch)
treed7e6c0431f0e96313f968d6a0c047f0d0568a34d
parent693eabb2be4a57f1300e6b73d6d8069b355782c6 (diff)
compile.c: not flip-flop
* compile.c (iseq_compile_each): turn flip-flop in a not-operator into a boolean value. fix up r56315 git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@56521 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--ChangeLog5
-rw-r--r--compile.c15
-rw-r--r--test/ruby/test_flip.rb2
3 files changed, 22 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index aed05a54fc..4b5e01027d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Sat Oct 29 10:09:38 2016 Nobuyoshi Nakada <nobu@ruby-lang.org>
+
+ * compile.c (iseq_compile_each): turn flip-flop in a not-operator
+ into a boolean value. fix up r56315
+
Sat Oct 29 09:39:14 2016 Nobuyoshi Nakada <nobu@ruby-lang.org>
* vm_eval.c (vm_call0_body): follow the original class, not to
diff --git a/compile.c b/compile.c
index e183c688b5..bdeb7fdac2 100644
--- a/compile.c
+++ b/compile.c
@@ -5950,6 +5950,21 @@ iseq_compile_each(rb_iseq_t *iseq, LINK_ANCHOR *ret, NODE * node, int poped)
}
break;
}
+ case NODE_FLIP2:
+ case NODE_FLIP3:{
+ LABEL *lend = NEW_LABEL(line);
+ LABEL *ltrue = NEW_LABEL(line);
+ LABEL *lfalse = NEW_LABEL(line);
+ compile_branch_condition(iseq, ret, node, ltrue, lfalse);
+ ADD_INSNL(ret, line, jump, lend);
+ ADD_LABEL(ret, ltrue);
+ ADD_INSN1(ret, line, putobject, Qtrue);
+ ADD_INSNL(ret, line, jump, lend);
+ ADD_LABEL(ret, lfalse);
+ ADD_INSN1(ret, line, putobject, Qfalse);
+ ADD_LABEL(ret, lend);
+ break;
+ }
case NODE_SELF:{
if (!poped) {
ADD_INSN(ret, line, putself);
diff --git a/test/ruby/test_flip.rb b/test/ruby/test_flip.rb
index 7c805574a8..810fd5d3ae 100644
--- a/test/ruby/test_flip.rb
+++ b/test/ruby/test_flip.rb
@@ -8,6 +8,8 @@ class TestFlip < Test::Unit::TestCase
assert_equal [2], (1..9).select {|n| true if (n==2)..(n%2).zero?}
assert_equal [2,3,4], (1..9).select {|n| true if (n==2)...(n%2).zero?}
assert_equal [4,5,7,8], (1..9).select {|n| true if (n==4)...(n==5) or (n==7)...(n==8)}
+ assert_equal [nil, 2, 3, 4, nil], (1..5).map {|x| x if (x==2..x==4)}
+ assert_equal [1, nil, nil, nil, 5], (1..5).map {|x| x if !(x==2..x==4)}
end
def test_hidden_key