Fix for import_methods does not allow calling another refined method#9237
Fix for import_methods does not allow calling another refined method#9237headius merged 2 commits intojruby:masterfrom
Conversation
9dc3041 to
c729403
Compare
|
I'm trying to untangle this a bit so I understand the change. I'd like to better understand and document the conditions where a CallBase is potentiallyRefined but does not have a RefinedCachingCallSite. |
|
This fix works but only repairs the interpreter. JIT mode needs additional help to clear out already-compiled code when the scope's refinements are changed. I'm working on some additional patches on top of this PR. |
|
Interpreter and JIT can be fixed by having the import clear out old compile code, as @evaniainbrooks did for I also observed that the changes in |
|
I've pushed some additional changes as #9248, which fixes the JIT issue and eliminates the call site fix-up logic. The |
|
I will merge this PR and then my additional fixes. I'll open a separate issue for the |
issue #8876