kotovalexarian-likes-github/ruby--ruby
1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2022-11-09 12:17:21 -05:00
Code Releases Activity

YJIT doesn't need rb_obj_ensure_iv_index_mapping

Browse source 
We should make this function static and remove it from YJIT bindings.
This commit is contained in:
Aaron Patterson 2022-10-14 16:11:37 -07:00 committed by Aaron Patterson
parent 53e0e5e8df
commit 1acc1a5c6d
Notes: git 2022-10-15 09:15:04 +09:00 
Merged: https://github.com/ruby/ruby/pull/6553
4 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
internal/variable.h
View file

@ -35,7 +35,6 @@ void rb_gvar_ractor_local(const char *name);
static inline bool ROBJ_TRANSIENT_P(VALUE obj);
static inline void ROBJ_TRANSIENT_SET(VALUE obj);
static inline void ROBJ_TRANSIENT_UNSET(VALUE obj);
uint32_t rb_obj_ensure_iv_index_mapping(VALUE obj, ID id);
struct gen_ivtbl;
int rb_gen_ivtbl_get(VALUE obj, ID id, struct gen_ivtbl **ivtbl);

2
variable.c
View file

@ -1449,7 +1449,7 @@ rb_init_iv_list(VALUE obj)
// @note May raise when there are too many instance variables.
// @note YJIT uses this function at compile time to simplify the work needed to
// access the variable at runtime.
uint32_t
static uint32_t
rb_obj_ensure_iv_index_mapping(VALUE obj, ID id)
{
RUBY_ASSERT(RB_TYPE_P(obj, T_OBJECT));

1
yjit/bindgen/src/main.rs
View file

@ -304,7 +304,6 @@ fn main() {
// From internal/variable.h
.allowlist_function("rb_gvar_(get|set)")
.allowlist_function("rb_obj_ensure_iv_index_mapping")
// From include/ruby/internal/intern/variable.h
.allowlist_function("rb_attr_get")

3
yjit/src/cruby_bindings.inc.rs
View file

@ -1015,9 +1015,6 @@ extern "C" {
extern "C" {
pub fn rb_hash_resurrect(hash: VALUE) -> VALUE;
}
extern "C" {
pub fn rb_obj_ensure_iv_index_mapping(obj: VALUE, id: ID) -> u32;
}
extern "C" {
pub fn rb_gvar_get(arg1: ID) -> VALUE;
}