python / python/cpython

Tier Two Optimizer Eliminate Function Version Guards

Offen
#119,476 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

type-feature
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
35.9k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

Feature or enhancement

Proposal:

Suggested by @brandtbucher at the PyCon sprints

The tier two interpreter currently adds additional function function guards when it doesn't need to. We can use the function watcher to track when the function version changes and only add the first one.

Similar to the issue for type version guards https://github.com/python/cpython/issues/119258 and the implemented solution https://github.com/python/cpython/pull/119365

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

Linked PRs
  • gh-119491
  • gh-119510

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit der Durchsicht des zugehörigen type-version-guard-Issues 119258, seines implementierten PRs 119365 und der verknüpften PRs 119491 und 119510. Das beabsichtigte Ergebnis ist, dass der tier-two-Interpreter den function watcher verwendet, sodass er nur den ersten function version guard anstelle redundanter Guards hinzufügt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
compilers, performance
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.