Project

General

Profile

Actions

Feature #15305

open

[PATCH] include/ruby/ruby.h (RB_ALLOCV*): remove GC guard

Added by normalperson (Eric Wong) over 5 years ago. Updated over 2 years ago.

Status:
Open
Target version:
-
[ruby-core:89805]

Description

include/ruby/ruby.h (RB_ALLOCV*): remove unnecessary GC guard

At least I'm not seeing how it's necessary, as it defeats
optimization at allocation time (instead of protecting the
VALUE from being GC-ed)

Anyways, there's a measurable size reduction:

text           data     bss     dec     hex filename

3535061 21232 72944 3629237 3760b5 ruby.before
3534141 21232 72944 3628317 375d1d ruby.after

I also don't get the point of using atomics in rb_free_tmp_buffer;
but that's a separate issue...


Files

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0