From 7084c015bf09fbe1687f51953fce0f96fc48dab9 Mon Sep 17 00:00:00 2001
From: nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Thu, 5 Apr 2018 13:13:00 +0000
Subject: compile.c: fix positions encode

* compile.c (ibf_load_iseq_each): iseq_size necessary to encode
  positions is set in ibf_load_code().  [Bug #14660]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@63103 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 KNOWNBUGS.rb           | 22 ----------------------
 compile.c              |  6 +++---
 test/ruby/test_iseq.rb | 29 ++++++++++++++++++++++++-----
 3 files changed, 27 insertions(+), 30 deletions(-)

diff --git a/KNOWNBUGS.rb b/KNOWNBUGS.rb
index 0eedc340f3..35a8e75876 100644
--- a/KNOWNBUGS.rb
+++ b/KNOWNBUGS.rb
@@ -5,25 +5,3 @@
 # This test file includes tests which point out known bugs.
 # So all tests will cause failure.
 #
-assert_normal_exit("#{<<~"begin;"}\n#{<<~'end;#1'}", timeout: 5)
-begin;
-  str = "#{<<~"begin;"}\n#{<<~'end;'}"
-  begin;
-    class P
-      def p; end
-      def q; end
-      E = ""
-      N = "#{E}"
-      attr_reader :i
-      undef p
-      undef q
-      remove_const :E
-      remove_const :N
-    end
-  end;
-  iseq = RubyVM::InstructionSequence.compile(str)
-  100.times {|i|
-    bin = iseq.to_binary
-    RubyVM::InstructionSequence.load_from_binary(bin).eval
-  }
-end;#1
diff --git a/compile.c b/compile.c
index db02415ca5..a0bcb0c928 100644
--- a/compile.c
+++ b/compile.c
@@ -8870,15 +8870,15 @@ ibf_load_iseq_each(const struct ibf_load *load, rb_iseq_t *iseq, ibf_offset_t of
     load_body->param.keyword        = ibf_load_param_keyword(load, body);
     load_body->insns_info.body      = ibf_load_insns_info_body(load, body);
     load_body->insns_info.positions = ibf_load_insns_info_positions(load, body);
-#if VM_INSN_INFO_TABLE_IMPL == 2
-    rb_iseq_insns_info_encode_positions(iseq);
-#endif
     load_body->local_table          = ibf_load_local_table(load, body);
     load_body->catch_table          = ibf_load_catch_table(load, body);
     load_body->parent_iseq          = ibf_load_iseq(load, body->parent_iseq);
     load_body->local_iseq           = ibf_load_iseq(load, body->local_iseq);
 
     ibf_load_code(load, iseq, body);
+#if VM_INSN_INFO_TABLE_IMPL == 2
+    rb_iseq_insns_info_encode_positions(iseq);
+#endif
 
     rb_iseq_translate_threaded_code(iseq);
 }
diff --git a/test/ruby/test_iseq.rb b/test/ruby/test_iseq.rb
index 315771db3d..2c8acb6421 100644
--- a/test/ruby/test_iseq.rb
+++ b/test/ruby/test_iseq.rb
@@ -396,18 +396,37 @@ class TestISeq < Test::Unit::TestCase
     }
   end
 
-  def test_to_binary_with_objects
+  def assert_iseq_to_binary(code, mesg = nil)
     skip "does not work on other than x86" unless /x(?:86|64)|i\d86/ =~ RUBY_PLATFORM
-    code = "[]"+100.times.map{|i|"<</#{i}/"}.join
     iseq = RubyVM::InstructionSequence.compile(code)
-    bin = assert_nothing_raised do
+    bin = assert_nothing_raised(mesg) do
       iseq.to_binary
     rescue RuntimeError => e
       skip e.message if /compile with coverage/ =~ e.message
       raise
     end
-    skip "trace events does not load correctly since r62851"
+    10.times do
+      assert_equal(bin, iseq.to_binary, mesg)
+    end
     iseq2 = RubyVM::InstructionSequence.load_from_binary(bin)
-    assert_equal(iseq.to_a, iseq2.to_a)
+    assert_equal(iseq.to_a, iseq2.to_a, mesg)
+    iseq2
+  end
+
+  def test_to_binary_with_objects
+    assert_iseq_to_binary("[]"+100.times.map{|i|"<</#{i}/"}.join)
+  end
+
+  def test_to_binary_line_info
+    assert_iseq_to_binary("#{<<~"begin;"}\n#{<<~'end;'}", '[Bug #14660]').eval
+    begin;
+      class P
+        def p; end
+        def q; end
+        E = ""
+        N = "#{E}"
+        attr_reader :i
+      end
+    end;
   end
 end
-- 
cgit v1.2.3