-
Notifications
You must be signed in to change notification settings - Fork 25
[Compat][3.11] support call breakgraph #369
Conversation
Thanks for your contribution! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
其余没什么问题了~
喔喔 PR 描述里最好写详细一些,还是蛮有用的,比如之前和震哥讨论的时候也是翻 PR 回忆了下 NullVariable 相关的改动(#159),震哥 PR 描述一般写的都比较清晰的,包括整个 PR 的动机和具体改动,也能帮助新同学来熟悉整个项目(比如我在看到动转静代码里如果有某些奇怪的地方就会去翻当时的 PR) |
Co-authored-by: Nyakku Shigure <[email protected]>
Co-authored-by: Nyakku Shigure <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMeow 🐾
改完啦 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! LGTMeow 🐾
本PR改动如下:
-1
,因此实现自定义的calc_stack_effect计算 stack_effect值,也可以为后续其他版本适配起到一定帮助。