_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

Inizia con l’operazione tier2 _CHECK_FUNCTION mostrata nell’issue ed esamina come viene usato frame->f_funcobj quando le funzioni vengono incorporate. Esamina la PR collegata gh-126856 e determina se la versione della funzione può essere incorporata nell’istruzione senza interferire con la valutazione parziale; il lavoro è completato quando il controllo della versione non dipende più, ove possibile, dal riferimento al frame invalidato.