diff options
author | ko1 <ko1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-08-24 06:31:15 +0000 |
---|---|---|
committer | ko1 <ko1@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-08-24 06:31:15 +0000 |
commit | 7049d9c80d1ba859beb5d68c7a9de37d5c11b8e8 (patch) | |
tree | e64aa3d7290f296d3685aa27a3e0997fac84a76d /vm_insnhelper.c | |
parent | 4f03f0cb67b0bf6497ffd4051e872905e3b49ead (diff) | |
download | ruby-7049d9c80d1ba859beb5d68c7a9de37d5c11b8e8.tar.gz |
* iseq.h, iseq.c, compile.c: Change the line number data structure
to solve an issue reported at [ruby-dev:44413] [Ruby 1.9 - Bug #5217].
Before this fix, each instruction has an information including
line number (iseq::iseq_insn_info_table). Instead of this data
structure, recording only line number changing places
(iseq::iseq_line_info_table).
The order of entries in iseq_line_info_table is ascending order of
iseq_line_info_table_entry::position. You can get a line number
by an iseq and a program counter with this data structure.
This fix reduces memory consumption of iseq (bytecode).
On my measurement, a rails application consumes 21.8MB for
iseq with this fix on the 32bit CPU. Without this fix, it
consumes 24.7MB for iseq [ruby-dev:44415].
* proc.c: ditto.
* vm_insnhelper.c: ditto.
* vm_method.c: ditto.
* vm.c (rb_vm_get_sourceline): change to use rb_iseq_line_no().
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@33046 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'vm_insnhelper.c')
-rw-r--r-- | vm_insnhelper.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vm_insnhelper.c b/vm_insnhelper.c index a35dbb1196..918c7fb7de 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -118,8 +118,8 @@ argument_error(const rb_iseq_t *iseq, int miss_argc, int correct_argc) if (iseq) { int line_no = 1; - if (iseq->insn_info_size) { - line_no = iseq->insn_info_table[0].line_no; + if (iseq->line_info_size) { + line_no = iseq->line_info_table[0].line_no; } err_line = rb_sprintf("%s:%d:in `%s'", |