`LOAD_FAST_BORROW` not being used even when safe to do so, if value is live at BB end.
未关闭
还没有人认领这个 Issue。
3.15
interpreter-core
performance
type-feature
- 主要语言
- Python
- 星标
- 77.2k
- 派生
- 35.9k
- PR 合并指标
- PR 指标待抓取
描述
This function
def f(x, y, c):
return 1 + (x if c else y)
compiles to
1 RESUME 0
2 LOAD_SMALL_INT 1
LOAD_FAST_BORROW 2 (c)
TO_BOOL
POP_JUMP_IF_FALSE 9 (to L1)
NOT_TAKEN
LOAD_FAST_BORROW 0 (x)
BINARY_OP 0 (+)
RETURN_VALUE
L1: LOAD_FAST 1 (y)
BINARY_OP 0 (+)
RETURN_VALUE
Note that the load of y uses LOAD_FAST even though LOAD_FAST_BORROW is safe.
This becomes important with virtual iterators as the iterable for the loop is live at BB end.
Linked PRs
- gh-133721
- gh-148999
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
首先复现针对 f(x, y, c) 报告的反汇编,并调查编译器对在基本块末尾仍然存活的值的处理。比较条件分支和虚拟迭代器的情况,然后验证在安全的情况下 y 使用 LOAD_FAST_BORROW,并为所报告的行为添加或更新覆盖率。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- python
- 领域
- compilers
- Issue 类型
- 缺陷
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100