Add line_count field to rb_ast_body_t

This patch adds `int line_count` field to `rb_ast_body_t` structure. Instead, we no longer cast `script_lines` to Fixnum. ## Background Ref https://github.com/ruby/ruby/pull/10618 In the PR above, we have decoupled IMEMO from `rb_ast_t`. This means we could lift the five-words-restriction of the structure that forced us to unionize `rb_ast_t *` and `FIXNUM` in one field. ## Relating refactor - Remove the second parameter of `rb_ruby_ast_new()` function ## Attention I will remove a code that assigns -1 to line_count, in `rb_binding_add_dynavars()` of vm.c, because I don't think it is necessary. But I will make another PR for this so that we can atomically revert in case I was wrong (See the comment on the code) 

1 files changed, 2 insertions, 2 deletions

diff --git a/compile.c b/compile.c
index 077770a7e4..588ce0c7ee 100644
--- a/compile.c
+++ b/compile.c
@@ -1479,7 +1479,7 @@ new_child_iseq(rb_iseq_t *iseq, const NODE *const node,
 VALUE name, const rb_iseq_t *parent, enum rb_iseq_type type, int line_no)
 {
 rb_iseq_t *ret_iseq;
- VALUE vast = rb_ruby_ast_new(node, NULL);
+ VALUE vast = rb_ruby_ast_new(node);

 debugs("[new_child_iseq]> ---------------------------------------\n");
 int isolated_depth = ISEQ_COMPILE_DATA(iseq)->isolated_depth;
@@ -8771,7 +8771,7 @@ compile_builtin_mandatory_only_method(rb_iseq_t *iseq, const NODE *node, const N
 scope_node.nd_body = mandatory_node(iseq, node);
 scope_node.nd_args = &args_node;

- VALUE vast = rb_ruby_ast_new(RNODE(&scope_node), NULL);
+ VALUE vast = rb_ruby_ast_new(RNODE(&scope_node));

 ISEQ_BODY(iseq)->mandatory_only_iseq =
 rb_iseq_new_with_opt(vast, rb_iseq_base_label(iseq),