JIT: remove all uses of _PyType_LookupByVersion in optimizer_bytecodes.c
オープン
まだ誰も着手していません。
interpreter-core
topic-JIT
type-feature
type-refactor
- 主要言語
- Python
- スター
- 77.2k
- フォーク
- 35.9k
- PR マージ指標
- PR 指標を取得中
説明
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
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
optimizer_bytecodes.c から始め、リンクされたコミットと PR 148277 を記録値ガードの文脈として使用し、残っている _CHECK_ATTR_CLASS による _PyType_LookupByVersion の使用箇所を調べます。Issue に関連する作業が記載されているため、PR 148394 を確認します。残っている一覧上の操作が変換され、関連するテストが成功すれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- c, python
- 領域
- compilers
- issue の種類
- 機能追加
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100