The `LOAD_ATTR_INSTANCE_VALUE` instruction can be sped up by storing the offset of the `valid` field in the cache.
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 35.9k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
LOAD_ATTR_INSTANCE_VALUE needs to check the validity of the objects embedded values array.
Currently, it does that by computing the offset of the valid byte.
Instead we could store the offset in the inline cache as LOAD_ATTR_INSTANCE_VALUE has several inline cache entries to spare.
The check is done in the _CHECK_MANAGED_OBJECT_HAS_VALUES uop.
Linked PRs
- gh-153122
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der Anweisung LOAD_ATTR_INSTANCE_VALUE und dem uop _CHECK_MANAGED_OBJECT_HAS_VALUES in der Interpreter-Implementierung von CPython. Überprüfe, wie die Inline-Cache-Einträge der Anweisung definiert sind und wie der Offset des Validitätsbytes derzeit berechnet wird. Als abgeschlossen gilt die Arbeit, wenn der Offset in einem verfügbaren Inline-Cache-Eintrag gespeichert wird und die Validitätsprüfung ihn verwendet, wobei die zugehörigen Tests und Benchmarks weiterhin erfolgreich sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- performance
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100