diff options
| author | John Hawthorn <john@hawthorn.email> | 2023-04-19 10:40:40 -0700 |
|---|---|---|
| committer | John Hawthorn <john@hawthorn.email> | 2023-04-20 16:09:16 -0700 |
| commit | 072ef7a1aad9c2a9296d1df29996da4068c50b94 (patch) | |
| tree | d7652a39ed438c49893b9975f74ce332580b3721 /yjit | |
| parent | a42c36c0f1b0439826b37c868fac4ded41e7aa91 (diff) | |
| download | ruby-072ef7a1aad9c2a9296d1df29996da4068c50b94.tar.gz | |
YJIT: invokesuper: Remove cme mid matching check
This check was introduced to match an assertion in the C YJIT when this
was originally introduced. I don't believe it's necessary for
correctness of the generated code.
Co-authored-by: Adam Hess <HParker@github.com>
Co-authored-by: Daniel Colson <danieljamescolson@gmail.com>
Co-authored-by: Luan Vieira <luanzeba@github.com>
Diffstat (limited to 'yjit')
| -rw-r--r-- | yjit/src/codegen.rs | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/yjit/src/codegen.rs b/yjit/src/codegen.rs index 10544ea902..280b4e2c4c 100644 --- a/yjit/src/codegen.rs +++ b/yjit/src/codegen.rs @@ -7034,13 +7034,6 @@ fn gen_invokesuper( let current_defined_class = unsafe { (*me).defined_class }; let mid = unsafe { get_def_original_id((*me).def) }; - if me != unsafe { rb_callable_method_entry(current_defined_class, (*me).called_id) } { - // Though we likely could generate this call, as we are only concerned - // with the method entry remaining valid, assume_method_lookup_stable - // below requires that the method lookup matches as well - return None; - } - // vm_search_normal_superclass let rbasic_ptr: *const RBasic = current_defined_class.as_ptr(); if current_defined_class.builtin_type() == RUBY_T_ICLASS |