Redundant stack spills on subsequent escaping calls separated by comments
オープン
まだ誰も着手していません。
interpreter-core
type-bug
- 主要言語
- Python
- スター
- 77.2k
- フォーク
- 35.9k
- PR マージ指標
- PR 指標を取得中
説明
Bug report
Bug description:
Tier 1 case generator emits stack spill for every subsequent escaping call, when subsequent escaping calls are separated by comment(s). Example:
Subsequent escaping calls without comments in between:
Bytecode definition:
pure inst(NOP, (--)) {
escaping_call1();
escaping_call2();
escaping_call3();
}
Generated case:
TARGET(NOP) {
frame->instr_ptr = next_instr;
next_instr += 1;
INSTRUCTION_STATS(NOP);
_PyFrame_SetStackPointer(frame, stack_pointer);
escaping_call1();
escaping_call2();
escaping_call3();
stack_pointer = _PyFrame_GetStackPointer(frame);
DISPATCH();
}
Subsequent escaping calls with comments in between:
Bytecode definition:
pure inst(NOP, (--)) {
escaping_call1();
// comment1
escaping_call2();
// comment2
escaping_call3();
}
Generated case:
TARGET(NOP) {
frame->instr_ptr = next_instr;
next_instr += 1;
INSTRUCTION_STATS(NOP);
_PyFrame_SetStackPointer(frame, stack_pointer);
escaping_call1();
stack_pointer = _PyFrame_GetStackPointer(frame);
// comment1
_PyFrame_SetStackPointer(frame, stack_pointer);
escaping_call2();
stack_pointer = _PyFrame_GetStackPointer(frame);
// comment2
_PyFrame_SetStackPointer(frame, stack_pointer);
escaping_call3();
stack_pointer = _PyFrame_GetStackPointer(frame);
DISPATCH();
}
CPython versions tested on:
CPython main branch
Operating systems tested on:
No response
Linked PRs
- gh-128761
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、レポートに示されている Tier 1 case generator と bytecode definition から始め、次に現在の作業についてリンク先の PR gh-128761 を確認してください。escaping 呼び出しの間にコメントを入れて例を再現し、生成されたケースが必要な stack-pointer 操作だけを出力することを確認してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- c, python
- 領域
- compilers
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 25/100