JIT: remove all uses of _PyType_LookupByVersion in optimizer_bytecodes.c
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 35.9k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Feature or enhancement
Proposal:
We should use recorded values instead, as that is not bounded by a limited-size cache, and it will be easier to make the optimizer thread safe for FT. See for example how I use recorded values for guards in this commit https://github.com/python/cpython/pull/148277/commits/8fab9d33c998d9c50a85f7705d0205aa175d03ab
Ops that need converting:
- _CHECK_ATTR_CLASS
- _GUARD_TYPE_VERSION
@NekoAsakura would you like to work on this?
Has this already been discussed elsewhere?
No response given
Links to previous discussion of this feature:
No response
Linked PRs
- gh-148394
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez dans optimizer_bytecodes.c et examinez l’utilisation restante de _PyType_LookupByVersion par _CHECK_ATTR_CLASS, en utilisant le commit lié et la PR 148277 comme contexte pour les gardes de valeurs enregistrées. Vérifiez la PR 148394, car l’issue répertorie du travail lié. C’est terminé lorsque l’opération restante répertoriée est convertie et que ses tests pertinents passent.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- c, python
- Domaine
- compilers
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100