_CHECK_FUNCTION interferes with partial evaluation

Feature or enhancement

Proposal:

A reminder of how _CHECK_FUNCTION looks like:

        tier2 op(_CHECK_FUNCTION, (func_version/2 -- )) {
            assert(PyStackRef_FunctionCheck(frame->f_funcobj));
            PyFunctionObject *func = (PyFunctionObject *)PyStackRef_AsPyObjectBorrow(frame->f_funcobj);
            DEOPT_IF(func->func_version != func_version);
        }

frame->f_funcobj becomes invalid once we start inlining functions. Thus we should "burn in" the function object version to be checked whenever possible, to not interfere with inlining and partial evaluation.

An added benefit is that the "burned in" version is also cheaper as it doesn't have to read from the frame struct, rather reading from the instruction itself in the JIT.

Has this already been discussed elsewhere?

No response given

Links to previous discussion of this feature:

No response

Linked PRs

• gh-126856

Commencez par l’opération tier2 _CHECK_FUNCTION présentée dans l’issue et examinez comment frame->f_funcobj est utilisé lorsque les fonctions sont intégrées. Examinez la PR liée gh-126856 et déterminez si la version de la fonction peut être intégrée à l’instruction sans interférer avec l’évaluation partielle ; le travail est considéré comme terminé lorsque la vérification de version ne dépend plus, lorsque c’est possible, de la référence de frame invalidée.