mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
Revert "Manage AST NODEs out of GC"
This reverts commit 620ba74778
.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60486 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
620ba74778
commit
5a176b75b1
10 changed files with 68 additions and 250 deletions
7
gc.c
7
gc.c
|
@ -434,7 +434,6 @@ typedef struct RVALUE {
|
|||
const rb_iseq_t iseq;
|
||||
rb_env_t env;
|
||||
struct rb_imemo_alloc_struct alloc;
|
||||
ast_t ast;
|
||||
} imemo;
|
||||
struct {
|
||||
struct RBasic basic;
|
||||
|
@ -2360,9 +2359,6 @@ obj_free(rb_objspace_t *objspace, VALUE obj)
|
|||
case imemo_alloc:
|
||||
xfree(RANY(obj)->as.imemo.alloc.ptr);
|
||||
break;
|
||||
case imemo_ast:
|
||||
rb_ast_free(&RANY(obj)->as.imemo.ast);
|
||||
break;
|
||||
default:
|
||||
break;
|
||||
}
|
||||
|
@ -4544,9 +4540,6 @@ gc_mark_imemo(rb_objspace_t *objspace, VALUE obj)
|
|||
} while ((m = m->next) != NULL);
|
||||
}
|
||||
return;
|
||||
case imemo_ast:
|
||||
rb_ast_mark(&RANY(obj)->as.imemo.ast);
|
||||
return;
|
||||
#if VM_CHECK_MODE > 0
|
||||
default:
|
||||
VM_UNREACHABLE(gc_mark_imemo);
|
||||
|
|
|
@ -844,8 +844,7 @@ enum imemo_type {
|
|||
imemo_memo = 5,
|
||||
imemo_ment = 6,
|
||||
imemo_iseq = 7,
|
||||
imemo_alloc = 8,
|
||||
imemo_ast = 9
|
||||
imemo_alloc = 8
|
||||
};
|
||||
#define IMEMO_MASK 0x0f
|
||||
|
||||
|
|
35
iseq.c
35
iseq.c
|
@ -641,9 +641,9 @@ rb_iseq_compile_with_option(VALUE src, VALUE file, VALUE realpath, VALUE line, c
|
|||
#else
|
||||
# define INITIALIZED /* volatile */
|
||||
#endif
|
||||
ast_t *(*parse)(VALUE vparser, VALUE fname, VALUE file, int start);
|
||||
NODE *(*parse)(VALUE vparser, VALUE fname, VALUE file, int start);
|
||||
int ln;
|
||||
ast_t *INITIALIZED ast;
|
||||
NODE *INITIALIZED node;
|
||||
|
||||
/* safe results first */
|
||||
make_compile_option(&option, opt);
|
||||
|
@ -659,20 +659,18 @@ rb_iseq_compile_with_option(VALUE src, VALUE file, VALUE realpath, VALUE line, c
|
|||
{
|
||||
const VALUE parser = rb_parser_new();
|
||||
rb_parser_set_context(parser, base_block, FALSE);
|
||||
ast = (*parse)(parser, file, src, ln);
|
||||
node = (*parse)(parser, file, src, ln);
|
||||
}
|
||||
|
||||
if (!ast->root) {
|
||||
rb_ast_dispose(ast);
|
||||
if (!node) {
|
||||
rb_exc_raise(th->ec->errinfo);
|
||||
}
|
||||
else {
|
||||
INITIALIZED VALUE label = parent ?
|
||||
parent->body->location.label :
|
||||
rb_fstring_cstr("<compiled>");
|
||||
iseq = rb_iseq_new_with_opt(ast->root, label, file, realpath, line,
|
||||
iseq = rb_iseq_new_with_opt(node, label, file, realpath, line,
|
||||
parent, type, &option);
|
||||
rb_ast_dispose(ast);
|
||||
}
|
||||
|
||||
return iseq;
|
||||
|
@ -853,8 +851,8 @@ static VALUE
|
|||
iseqw_s_compile_file(int argc, VALUE *argv, VALUE self)
|
||||
{
|
||||
VALUE file, line = INT2FIX(1), opt = Qnil;
|
||||
VALUE parser, f, exc = Qnil, ret;
|
||||
ast_t *ast;
|
||||
VALUE parser, f, exc = Qnil;
|
||||
const NODE *node;
|
||||
rb_compile_option_t option;
|
||||
int i;
|
||||
|
||||
|
@ -871,23 +869,18 @@ iseqw_s_compile_file(int argc, VALUE *argv, VALUE self)
|
|||
|
||||
parser = rb_parser_new();
|
||||
rb_parser_set_context(parser, NULL, FALSE);
|
||||
ast = rb_parser_compile_file_path(parser, file, f, NUM2INT(line));
|
||||
if (!ast->root) exc = GET_EC()->errinfo;
|
||||
node = rb_parser_compile_file_path(parser, file, f, NUM2INT(line));
|
||||
if (!node) exc = GET_EC()->errinfo;
|
||||
|
||||
rb_io_close(f);
|
||||
if (!ast->root) {
|
||||
rb_ast_dispose(ast);
|
||||
rb_exc_raise(exc);
|
||||
}
|
||||
if (!node) rb_exc_raise(exc);
|
||||
|
||||
make_compile_option(&option, opt);
|
||||
|
||||
ret = iseqw_new(rb_iseq_new_with_opt(ast->root, rb_fstring_cstr("<main>"),
|
||||
file,
|
||||
rb_realpath_internal(Qnil, file, 1),
|
||||
line, NULL, ISEQ_TYPE_TOP, &option));
|
||||
rb_ast_dispose(ast);
|
||||
return ret;
|
||||
return iseqw_new(rb_iseq_new_with_opt(node, rb_fstring_cstr("<main>"),
|
||||
file,
|
||||
rb_realpath_internal(Qnil, file, 1),
|
||||
line, NULL, ISEQ_TYPE_TOP, &option));
|
||||
}
|
||||
|
||||
/*
|
||||
|
|
7
load.c
7
load.c
|
@ -602,7 +602,7 @@ rb_load_internal0(rb_thread_t *th, VALUE fname, int wrap)
|
|||
EC_PUSH_TAG(th->ec);
|
||||
state = EXEC_TAG();
|
||||
if (state == TAG_NONE) {
|
||||
ast_t *ast;
|
||||
NODE *node;
|
||||
const rb_iseq_t *iseq;
|
||||
|
||||
if ((iseq = rb_iseq_load_iseq(fname)) != NULL) {
|
||||
|
@ -611,10 +611,9 @@ rb_load_internal0(rb_thread_t *th, VALUE fname, int wrap)
|
|||
else {
|
||||
VALUE parser = rb_parser_new();
|
||||
rb_parser_set_context(parser, NULL, FALSE);
|
||||
ast = (ast_t *)rb_parser_load_file(parser, fname);
|
||||
iseq = rb_iseq_new_top(ast->root, rb_fstring_cstr("<top (required)>"),
|
||||
node = (NODE *)rb_parser_load_file(parser, fname);
|
||||
iseq = rb_iseq_new_top(node, rb_fstring_cstr("<top (required)>"),
|
||||
fname, rb_realpath_internal(Qnil, fname, 1), NULL);
|
||||
rb_ast_dispose(ast);
|
||||
}
|
||||
rb_iseq_eval(iseq);
|
||||
}
|
||||
|
|
104
node.c
104
node.c
|
@ -1211,107 +1211,3 @@ rb_gc_mark_node(NODE *obj)
|
|||
}
|
||||
return 0;
|
||||
}
|
||||
|
||||
typedef struct node_buffer_elem_struct {
|
||||
struct node_buffer_elem_struct *next;
|
||||
NODE buf[1];
|
||||
} node_buffer_elem_t;
|
||||
|
||||
typedef struct node_buffer_struct {
|
||||
long idx, len;
|
||||
node_buffer_elem_t *head;
|
||||
node_buffer_elem_t body;
|
||||
} node_buffer_t;
|
||||
|
||||
node_buffer_t *
|
||||
rb_node_buffer_new()
|
||||
{
|
||||
node_buffer_t *nb = xmalloc(sizeof(node_buffer_t) + 16 * sizeof(NODE));
|
||||
nb->idx = 0;
|
||||
nb->len = 16;
|
||||
nb->head = &nb->body;
|
||||
nb->head->next = NULL;
|
||||
return nb;
|
||||
}
|
||||
|
||||
void
|
||||
rb_node_buffer_free(node_buffer_t *nb)
|
||||
{
|
||||
node_buffer_elem_t *nbe = nb->head;
|
||||
|
||||
while (nbe != &nb->body) {
|
||||
void *buf = nbe;
|
||||
nbe = nbe->next;
|
||||
xfree(buf);
|
||||
}
|
||||
xfree(nb);
|
||||
}
|
||||
|
||||
NODE *
|
||||
rb_ast_newnode(ast_t *ast)
|
||||
{
|
||||
node_buffer_t *nb = ast->node_buffer;
|
||||
if (nb->idx >= nb->len) {
|
||||
long n = nb->len * 2;
|
||||
node_buffer_elem_t *nbe;
|
||||
nbe = xmalloc(sizeof(node_buffer_elem_t) + n * sizeof(NODE));
|
||||
nb->idx = 0;
|
||||
nb->len = n;
|
||||
nbe->next = nb->head;
|
||||
nb->head = nbe;
|
||||
}
|
||||
return &nb->head->buf[nb->idx++];
|
||||
}
|
||||
|
||||
void
|
||||
rb_ast_delete_node(ast_t *ast, NODE *n)
|
||||
{
|
||||
(void)ast;
|
||||
(void)n;
|
||||
/* should we implement freelist? */
|
||||
}
|
||||
|
||||
ast_t *
|
||||
rb_ast_new(void)
|
||||
{
|
||||
return (ast_t *)rb_imemo_new(imemo_ast, 0, (VALUE)rb_node_buffer_new(), rb_ary_tmp_new(0), 0);
|
||||
}
|
||||
|
||||
void
|
||||
rb_ast_mark(ast_t *ast)
|
||||
{
|
||||
if (ast->node_buffer) rb_gc_mark(ast->mark_ary);
|
||||
}
|
||||
|
||||
void
|
||||
rb_ast_free(ast_t *ast)
|
||||
{
|
||||
if (ast->node_buffer) rb_node_buffer_free(ast->node_buffer);
|
||||
ast->node_buffer = 0;
|
||||
ast->root = 0;
|
||||
ast->mark_ary = 0;
|
||||
}
|
||||
|
||||
void
|
||||
rb_ast_dispose(ast_t *ast)
|
||||
{
|
||||
rb_ast_free(ast);
|
||||
rb_gc_writebarrier_remember((VALUE)ast);
|
||||
}
|
||||
|
||||
void
|
||||
rb_ast_add_mark_object(ast_t *ast, VALUE obj)
|
||||
{
|
||||
rb_ary_push(ast->mark_ary, obj);
|
||||
}
|
||||
|
||||
void
|
||||
rb_ast_delete_mark_object(ast_t *ast, VALUE obj)
|
||||
{
|
||||
long i;
|
||||
for (i = 0; i < RARRAY_LEN(ast->mark_ary); i++) {
|
||||
if (obj == RARRAY_AREF(ast->mark_ary, i)) {
|
||||
RARRAY_ASET(ast->mark_ary, i, Qnil);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
34
node.h
34
node.h
|
@ -439,24 +439,6 @@ typedef struct RNode {
|
|||
|
||||
RUBY_SYMBOL_EXPORT_BEGIN
|
||||
|
||||
typedef struct node_buffer_struct node_buffer_t;
|
||||
/* T_IMEMO/ast */
|
||||
typedef struct ast_struct {
|
||||
VALUE flags;
|
||||
VALUE reserved1;
|
||||
NODE *root;
|
||||
node_buffer_t *node_buffer;
|
||||
VALUE mark_ary;
|
||||
} ast_t;
|
||||
ast_t *rb_ast_new();
|
||||
void rb_ast_mark(ast_t*);
|
||||
void rb_ast_dispose(ast_t*);
|
||||
void rb_ast_free(ast_t*);
|
||||
void rb_ast_add_mark_object(ast_t*, VALUE);
|
||||
void rb_ast_delete_mark_object(ast_t*, VALUE);
|
||||
NODE *rb_ast_newnode(ast_t*);
|
||||
void rb_ast_delete_node(ast_t*, NODE *n);
|
||||
|
||||
VALUE rb_parser_new(void);
|
||||
VALUE rb_parser_end_seen_p(VALUE);
|
||||
VALUE rb_parser_encoding(VALUE);
|
||||
|
@ -465,15 +447,15 @@ VALUE rb_parser_set_yydebug(VALUE, VALUE);
|
|||
VALUE rb_parser_dump_tree(NODE *node, int comment);
|
||||
void rb_parser_set_options(VALUE, int, int, int, int);
|
||||
|
||||
ast_t *rb_parser_compile_cstr(VALUE, const char*, const char*, int, int);
|
||||
ast_t *rb_parser_compile_string(VALUE, const char*, VALUE, int);
|
||||
ast_t *rb_parser_compile_file(VALUE, const char*, VALUE, int);
|
||||
ast_t *rb_parser_compile_string_path(VALUE vparser, VALUE fname, VALUE src, int line);
|
||||
ast_t *rb_parser_compile_file_path(VALUE vparser, VALUE fname, VALUE input, int line);
|
||||
NODE *rb_parser_compile_cstr(VALUE, const char*, const char*, int, int);
|
||||
NODE *rb_parser_compile_string(VALUE, const char*, VALUE, int);
|
||||
NODE *rb_parser_compile_file(VALUE, const char*, VALUE, int);
|
||||
NODE *rb_parser_compile_string_path(VALUE vparser, VALUE fname, VALUE src, int line);
|
||||
NODE *rb_parser_compile_file_path(VALUE vparser, VALUE fname, VALUE input, int line);
|
||||
|
||||
ast_t *rb_compile_cstr(const char*, const char*, int, int);
|
||||
ast_t *rb_compile_string(const char*, VALUE, int);
|
||||
ast_t *rb_compile_file(const char*, VALUE, int);
|
||||
NODE *rb_compile_cstr(const char*, const char*, int, int);
|
||||
NODE *rb_compile_string(const char*, VALUE, int);
|
||||
NODE *rb_compile_file(const char*, VALUE, int);
|
||||
|
||||
void rb_node_init(NODE *n, enum node_type type, VALUE a0, VALUE a1, VALUE a2);
|
||||
NODE *rb_node_newnode(enum node_type,VALUE,VALUE,VALUE);
|
||||
|
|
80
parse.y
80
parse.y
|
@ -239,7 +239,6 @@ struct parser_params {
|
|||
unsigned int do_chomp: 1;
|
||||
unsigned int do_split: 1;
|
||||
|
||||
ast_t *ast;
|
||||
NODE *eval_tree_begin;
|
||||
NODE *eval_tree;
|
||||
VALUE error_buffer;
|
||||
|
@ -339,26 +338,14 @@ parser_set_line(NODE *n, int l)
|
|||
}
|
||||
|
||||
static inline void
|
||||
rb_discard_node_gen(struct parser_params *parser, NODE *n)
|
||||
rb_discard_node(NODE *n)
|
||||
{
|
||||
#ifndef RIPPER
|
||||
rb_ast_delete_node(parser->ast, n);
|
||||
#else
|
||||
rb_gc_force_recycle((VALUE)n);
|
||||
#endif
|
||||
}
|
||||
#define rb_discard_node(n) rb_discard_node_gen(parser, (n))
|
||||
|
||||
#define add_mark_object(obj) (void)(obj)
|
||||
|
||||
#ifndef RIPPER
|
||||
static inline void
|
||||
add_mark_object_gen(struct parser_params *parser, VALUE obj)
|
||||
{
|
||||
if (!SPECIAL_CONST_P(obj)) {
|
||||
rb_ast_add_mark_object(parser->ast, obj);
|
||||
}
|
||||
}
|
||||
#define add_mark_object(obj) add_mark_object_gen(parser, (obj))
|
||||
|
||||
static inline void
|
||||
set_line_body(NODE *body, int line)
|
||||
{
|
||||
|
@ -588,8 +575,6 @@ static NODE *parser_heredoc_dedent(struct parser_params*,NODE*);
|
|||
#else /* RIPPER */
|
||||
#define NODE_RIPPER NODE_CDECL
|
||||
|
||||
#define add_mark_object(obj) (void)(obj)
|
||||
|
||||
static inline VALUE
|
||||
ripper_new_yylval(ID a, VALUE b, VALUE c)
|
||||
{
|
||||
|
@ -2865,7 +2850,6 @@ primary : literal
|
|||
break;
|
||||
}
|
||||
}
|
||||
add_mark_object((VALUE)rb_imemo_alloc_new((VALUE)tbl, 0, 0, 0));
|
||||
scope = NEW_NODE(NODE_SCOPE, tbl, $8, args);
|
||||
nd_set_column(scope, @1.first_column);
|
||||
tbl[0] = 1; tbl[1] = id;
|
||||
|
@ -5589,55 +5573,52 @@ lex_getline(struct parser_params *parser)
|
|||
static const rb_data_type_t parser_data_type;
|
||||
|
||||
#ifndef RIPPER
|
||||
static ast_t*
|
||||
static NODE*
|
||||
parser_compile_string(VALUE vparser, VALUE fname, VALUE s, int line)
|
||||
{
|
||||
struct parser_params *parser;
|
||||
ast_t *ast;
|
||||
NODE *node;
|
||||
|
||||
TypedData_Get_Struct(vparser, struct parser_params, &parser_data_type, parser);
|
||||
parser->ast = ast = rb_ast_new();
|
||||
|
||||
lex_gets = lex_get_str;
|
||||
lex_gets_ptr = 0;
|
||||
lex_input = rb_str_new_frozen(s);
|
||||
lex_pbeg = lex_p = lex_pend = 0;
|
||||
|
||||
ast->root = yycompile(parser, fname, line);
|
||||
parser->ast = 0;
|
||||
node = yycompile(parser, fname, line);
|
||||
RB_GC_GUARD(vparser); /* prohibit tail call optimization */
|
||||
|
||||
return ast;
|
||||
return node;
|
||||
}
|
||||
|
||||
ast_t*
|
||||
NODE*
|
||||
rb_compile_string(const char *f, VALUE s, int line)
|
||||
{
|
||||
must_be_ascii_compatible(s);
|
||||
return parser_compile_string(rb_parser_new(), rb_filesystem_str_new_cstr(f), s, line);
|
||||
}
|
||||
|
||||
ast_t*
|
||||
NODE*
|
||||
rb_parser_compile_string(VALUE vparser, const char *f, VALUE s, int line)
|
||||
{
|
||||
return rb_parser_compile_string_path(vparser, rb_filesystem_str_new_cstr(f), s, line);
|
||||
}
|
||||
|
||||
ast_t*
|
||||
NODE*
|
||||
rb_parser_compile_string_path(VALUE vparser, VALUE f, VALUE s, int line)
|
||||
{
|
||||
must_be_ascii_compatible(s);
|
||||
return parser_compile_string(vparser, f, s, line);
|
||||
}
|
||||
|
||||
ast_t*
|
||||
NODE*
|
||||
rb_compile_cstr(const char *f, const char *s, int len, int line)
|
||||
{
|
||||
VALUE str = rb_str_new(s, len);
|
||||
return parser_compile_string(rb_parser_new(), rb_filesystem_str_new_cstr(f), str, line);
|
||||
}
|
||||
|
||||
ast_t*
|
||||
NODE*
|
||||
rb_parser_compile_cstr(VALUE vparser, const char *f, const char *s, int len, int line)
|
||||
{
|
||||
VALUE str = rb_str_new(s, len);
|
||||
|
@ -5652,7 +5633,7 @@ lex_io_gets(struct parser_params *parser, VALUE io)
|
|||
return rb_io_gets_internal(io);
|
||||
}
|
||||
|
||||
ast_t*
|
||||
NODE*
|
||||
rb_compile_file(const char *f, VALUE file, int start)
|
||||
{
|
||||
VALUE vparser = rb_parser_new();
|
||||
|
@ -5660,30 +5641,27 @@ rb_compile_file(const char *f, VALUE file, int start)
|
|||
return rb_parser_compile_file(vparser, f, file, start);
|
||||
}
|
||||
|
||||
ast_t*
|
||||
NODE*
|
||||
rb_parser_compile_file(VALUE vparser, const char *f, VALUE file, int start)
|
||||
{
|
||||
return rb_parser_compile_file_path(vparser, rb_filesystem_str_new_cstr(f), file, start);
|
||||
}
|
||||
|
||||
ast_t*
|
||||
NODE*
|
||||
rb_parser_compile_file_path(VALUE vparser, VALUE fname, VALUE file, int start)
|
||||
{
|
||||
struct parser_params *parser;
|
||||
ast_t *ast;
|
||||
NODE *node;
|
||||
|
||||
TypedData_Get_Struct(vparser, struct parser_params, &parser_data_type, parser);
|
||||
parser->ast = ast = rb_ast_new();
|
||||
|
||||
lex_gets = lex_io_gets;
|
||||
lex_input = file;
|
||||
lex_pbeg = lex_p = lex_pend = 0;
|
||||
|
||||
ast->root = yycompile(parser, fname, start);
|
||||
parser->ast = 0;
|
||||
node = yycompile(parser, fname, start);
|
||||
RB_GC_GUARD(vparser); /* prohibit tail call optimization */
|
||||
|
||||
return ast;
|
||||
return node;
|
||||
}
|
||||
#endif /* !RIPPER */
|
||||
|
||||
|
@ -6182,11 +6160,8 @@ parser_regx_options(struct parser_params *parser)
|
|||
}
|
||||
|
||||
static void
|
||||
dispose_string(struct parser_params *parser, VALUE str)
|
||||
dispose_string(VALUE str)
|
||||
{
|
||||
#ifndef RIPPER
|
||||
rb_ast_delete_mark_object(parser->ast, str);
|
||||
#endif
|
||||
rb_str_free(str);
|
||||
rb_gc_force_recycle(str);
|
||||
}
|
||||
|
@ -6664,7 +6639,7 @@ parser_heredoc_restore(struct parser_params *parser, NODE *here)
|
|||
lex_p = lex_pbeg + here->nd_nth;
|
||||
heredoc_end = ruby_sourceline;
|
||||
ruby_sourceline = nd_line(here);
|
||||
dispose_string(parser, here->nd_lit);
|
||||
dispose_string(here->nd_lit);
|
||||
rb_discard_node(here);
|
||||
token_flush(parser);
|
||||
}
|
||||
|
@ -6950,7 +6925,7 @@ parser_here_document(struct parser_params *parser, NODE *here)
|
|||
}
|
||||
if (nextc() == -1) {
|
||||
if (str) {
|
||||
dispose_string(parser, str);
|
||||
dispose_string(str);
|
||||
str = 0;
|
||||
}
|
||||
goto error;
|
||||
|
@ -8825,10 +8800,7 @@ yylex(YYSTYPE *lval, YYLTYPE *yylloc, struct parser_params *parser)
|
|||
static NODE*
|
||||
node_newnode(struct parser_params *parser, enum node_type type, VALUE a0, VALUE a1, VALUE a2)
|
||||
{
|
||||
NODE *n = rb_ast_newnode(parser->ast);
|
||||
|
||||
rb_node_init(n, type, a0, a1, a2);
|
||||
|
||||
NODE *n = (rb_node_newnode)(type, a0, a1, a2);
|
||||
nd_set_line(n, ruby_sourceline);
|
||||
/* mark not cared column to -1 */
|
||||
nd_set_column(n, -1);
|
||||
|
@ -10618,7 +10590,6 @@ new_args_tail_gen(struct parser_params *parser, NODE *k, ID kr, ID b, int column
|
|||
NODE *node;
|
||||
|
||||
args = ZALLOC(struct rb_args_info);
|
||||
add_mark_object((VALUE)rb_imemo_alloc_new((VALUE)args, 0, 0, 0));
|
||||
node = NEW_NODE(NODE_ARGS, 0, 0, args);
|
||||
nd_set_column(node, column);
|
||||
if (parser->error_p) return node;
|
||||
|
@ -10988,9 +10959,6 @@ local_tbl_gen(struct parser_params *parser)
|
|||
}
|
||||
if (--j < cnt) REALLOC_N(buf, ID, (cnt = j) + 1);
|
||||
buf[0] = cnt;
|
||||
|
||||
add_mark_object((VALUE)rb_imemo_alloc_new((VALUE)buf, 0, 0, 0));
|
||||
|
||||
return buf;
|
||||
}
|
||||
#endif
|
||||
|
@ -11382,16 +11350,18 @@ parser_mark(void *ptr)
|
|||
{
|
||||
struct parser_params *parser = (struct parser_params*)ptr;
|
||||
|
||||
rb_gc_mark((VALUE)lex_strterm);
|
||||
rb_gc_mark(lex_input);
|
||||
rb_gc_mark(lex_lastline);
|
||||
rb_gc_mark(lex_nextline);
|
||||
rb_gc_mark(ruby_sourcefile_string);
|
||||
#ifndef RIPPER
|
||||
rb_gc_mark((VALUE)ruby_eval_tree_begin);
|
||||
rb_gc_mark((VALUE)ruby_eval_tree);
|
||||
rb_gc_mark(ruby_debug_lines);
|
||||
rb_gc_mark(parser->compile_option);
|
||||
rb_gc_mark(parser->error_buffer);
|
||||
#else
|
||||
rb_gc_mark((VALUE)lex_strterm);
|
||||
rb_gc_mark(parser->delayed);
|
||||
rb_gc_mark(parser->value);
|
||||
rb_gc_mark(parser->result);
|
||||
|
|
35
ruby.c
35
ruby.c
|
@ -177,7 +177,7 @@ cmdline_options_init(ruby_cmdline_options_t *opt)
|
|||
return opt;
|
||||
}
|
||||
|
||||
static ast_t *load_file(VALUE parser, VALUE fname, VALUE f, int script,
|
||||
static NODE *load_file(VALUE parser, VALUE fname, VALUE f, int script,
|
||||
ruby_cmdline_options_t *opt);
|
||||
static VALUE open_load_file(VALUE fname_v, int *xflag);
|
||||
static void forbid_setid(const char *, const ruby_cmdline_options_t *);
|
||||
|
@ -1461,7 +1461,7 @@ rb_f_chomp(int argc, VALUE *argv)
|
|||
static VALUE
|
||||
process_options(int argc, char **argv, ruby_cmdline_options_t *opt)
|
||||
{
|
||||
ast_t *ast = 0;
|
||||
NODE *tree = 0;
|
||||
VALUE parser;
|
||||
VALUE script_name;
|
||||
const rb_iseq_t *iseq;
|
||||
|
@ -1674,12 +1674,12 @@ process_options(int argc, char **argv, ruby_cmdline_options_t *opt)
|
|||
ruby_set_script_name(progname);
|
||||
rb_parser_set_options(parser, opt->do_print, opt->do_loop,
|
||||
opt->do_line, opt->do_split);
|
||||
ast = rb_parser_compile_string(parser, opt->script, opt->e_script, 1);
|
||||
tree = rb_parser_compile_string(parser, opt->script, opt->e_script, 1);
|
||||
}
|
||||
else {
|
||||
VALUE f;
|
||||
f = open_load_file(script_name, &opt->xflag);
|
||||
ast = load_file(parser, opt->script_name, f, 1, opt);
|
||||
tree = load_file(parser, opt->script_name, f, 1, opt);
|
||||
}
|
||||
ruby_set_script_name(opt->script_name);
|
||||
if (dump & DUMP_BIT(yydebug)) {
|
||||
|
@ -1704,10 +1704,7 @@ process_options(int argc, char **argv, ruby_cmdline_options_t *opt)
|
|||
rb_enc_set_default_internal(Qnil);
|
||||
rb_stdio_set_default_encoding();
|
||||
|
||||
if (!ast->root) {
|
||||
rb_ast_dispose(ast);
|
||||
return Qfalse;
|
||||
}
|
||||
if (!tree) return Qfalse;
|
||||
|
||||
process_sflag(&opt->sflag);
|
||||
opt->xflag = 0;
|
||||
|
@ -1726,13 +1723,10 @@ process_options(int argc, char **argv, ruby_cmdline_options_t *opt)
|
|||
}
|
||||
|
||||
if (dump & (DUMP_BIT(parsetree)|DUMP_BIT(parsetree_with_comment))) {
|
||||
rb_io_write(rb_stdout, rb_parser_dump_tree(ast->root, dump & DUMP_BIT(parsetree_with_comment)));
|
||||
rb_io_write(rb_stdout, rb_parser_dump_tree(tree, dump & DUMP_BIT(parsetree_with_comment)));
|
||||
rb_io_flush(rb_stdout);
|
||||
dump &= ~DUMP_BIT(parsetree)&~DUMP_BIT(parsetree_with_comment);
|
||||
if (!dump) {
|
||||
rb_ast_dispose(ast);
|
||||
return Qtrue;
|
||||
}
|
||||
if (!dump) return Qtrue;
|
||||
}
|
||||
|
||||
{
|
||||
|
@ -1746,8 +1740,7 @@ process_options(int argc, char **argv, ruby_cmdline_options_t *opt)
|
|||
#endif
|
||||
}
|
||||
base_block = toplevel_context(toplevel_binding);
|
||||
iseq = rb_iseq_new_main(ast->root, opt->script_name, path, vm_block_iseq(base_block));
|
||||
rb_ast_dispose(ast);
|
||||
iseq = rb_iseq_new_main(tree, opt->script_name, path, vm_block_iseq(base_block));
|
||||
}
|
||||
|
||||
if (dump & DUMP_BIT(insns)) {
|
||||
|
@ -1797,7 +1790,7 @@ load_file_internal(VALUE argp_v)
|
|||
ruby_cmdline_options_t *opt = argp->opt;
|
||||
VALUE f = argp->f;
|
||||
int line_start = 1;
|
||||
ast_t *ast = 0;
|
||||
NODE *tree = 0;
|
||||
rb_encoding *enc;
|
||||
ID set_encoding;
|
||||
|
||||
|
@ -1901,7 +1894,7 @@ load_file_internal(VALUE argp_v)
|
|||
return (VALUE)rb_parser_compile_string_path(parser, orig_fname, f, line_start);
|
||||
}
|
||||
rb_funcall(f, set_encoding, 2, rb_enc_from_encoding(enc), rb_str_new_cstr("-"));
|
||||
ast = rb_parser_compile_file_path(parser, orig_fname, f, line_start);
|
||||
tree = rb_parser_compile_file_path(parser, orig_fname, f, line_start);
|
||||
rb_funcall(f, set_encoding, 1, rb_parser_encoding(parser));
|
||||
if (script && rb_parser_end_seen_p(parser)) {
|
||||
/*
|
||||
|
@ -1919,7 +1912,7 @@ load_file_internal(VALUE argp_v)
|
|||
rb_define_global_const("DATA", f);
|
||||
argp->f = Qnil;
|
||||
}
|
||||
return (VALUE)ast;
|
||||
return (VALUE)tree;
|
||||
}
|
||||
|
||||
static VALUE
|
||||
|
@ -2011,7 +2004,7 @@ restore_load_file(VALUE arg)
|
|||
return Qnil;
|
||||
}
|
||||
|
||||
static ast_t *
|
||||
static NODE *
|
||||
load_file(VALUE parser, VALUE fname, VALUE f, int script, ruby_cmdline_options_t *opt)
|
||||
{
|
||||
struct load_file_arg arg;
|
||||
|
@ -2020,8 +2013,8 @@ load_file(VALUE parser, VALUE fname, VALUE f, int script, ruby_cmdline_options_t
|
|||
arg.script = script;
|
||||
arg.opt = opt;
|
||||
arg.f = f;
|
||||
return (ast_t *)rb_ensure(load_file_internal, (VALUE)&arg,
|
||||
restore_load_file, (VALUE)&arg);
|
||||
return (NODE *)rb_ensure(load_file_internal, (VALUE)&arg,
|
||||
restore_load_file, (VALUE)&arg);
|
||||
}
|
||||
|
||||
void *
|
||||
|
|
|
@ -121,14 +121,10 @@ prelude_eval(VALUE code, VALUE name, int line)
|
|||
FALSE, /* int debug_frozen_string_literal; */
|
||||
};
|
||||
|
||||
ast_t *ast = rb_parser_compile_string_path(rb_parser_new(), name, code, line);
|
||||
if (!ast->root) {
|
||||
rb_ast_dispose(ast);
|
||||
rb_exc_raise(rb_errinfo());
|
||||
}
|
||||
rb_iseq_eval(rb_iseq_new_with_opt(ast->root, name, name, Qnil, INT2FIX(line),
|
||||
NODE *node = rb_parser_compile_string_path(rb_parser_new(), name, code, line);
|
||||
if (!node) rb_exc_raise(rb_errinfo());
|
||||
rb_iseq_eval(rb_iseq_new_with_opt(node, name, name, Qnil, INT2FIX(line),
|
||||
NULL, ISEQ_TYPE_TOP, &optimization));
|
||||
rb_ast_dispose(ast);
|
||||
}
|
||||
% end
|
||||
|
||||
|
|
|
@ -290,9 +290,6 @@ class TestGc < Test::Unit::TestCase
|
|||
base_length = GC.stat[:heap_eden_pages]
|
||||
(base_length * 500).times{ 'a' }
|
||||
GC.start
|
||||
base_length = GC.stat[:heap_eden_pages]
|
||||
(base_length * 500).times{ 'a' }
|
||||
GC.start
|
||||
assert_in_epsilon base_length, (v = GC.stat[:heap_eden_pages]), 1/8r,
|
||||
"invalid heap expanding (base_length: #{base_length}, GC.stat[:heap_eden_pages]: #{v})"
|
||||
|
||||
|
|
Loading…
Reference in a new issue