_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

Comienza con la operación tier2 _CHECK_FUNCTION mostrada en el issue y examina cómo se usa frame->f_funcobj cuando las funciones se insertan inline. Revisa el PR vinculado gh-126856 y determina si la versión de la función puede incorporarse a la instrucción sin interferir con la evaluación parcial; se considera terminado cuando la comprobación de versión ya no depende, cuando sea posible, de la referencia al frame invalidada.