Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SOT][3.12] Support POP_JUMP_IF_NONE and POP_JUMP_IF_NOT_NONE opcode in Python 3.12 #62120

Conversation

diadestiny
Copy link
Contributor

@diadestiny diadestiny commented Feb 27, 2024

PR types

Others

PR changes

Others

Description

python 3.12 支持 POP_JUMP_IF_NONE and POP_JUMP_IF_NOT_NONE
test_11_jumps.py单测里面还涉及到“opcode: LOAD_FAST_CHECK is not supported.“等其他报错,暂不处理

Copy link

paddle-bot bot commented Feb 27, 2024

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Feb 27, 2024
SigureMo
SigureMo previously approved these changes Feb 27, 2024
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMeow

gouzil
gouzil previously approved these changes Feb 27, 2024
@SigureMo
Copy link
Member

需要解决一下冲突~

@diadestiny diadestiny dismissed stale reviews from gouzil and SigureMo via 6cfb270 February 27, 2024 12:48
SigureMo
SigureMo previously approved these changes Feb 27, 2024
@@ -1,9 +1,8 @@
./test_11_jumps.py
./test_12_for_loop.py
./test_21_global.py
./test_break_graph.py
./test_analysis_inputs.py
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这是又给加回来了???

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

看样子冲突解决错了 😂

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

SigureMo
SigureMo previously approved these changes Feb 27, 2024
gouzil
gouzil previously approved these changes Feb 28, 2024
@@ -1,9 +1,6 @@
./test_11_jumps.py
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LOAD_FAST_CHECK还没支持先不开吧

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你的 PR 里是不是支持了 LOAD_FAST_CHECK 并解决了 POP_JUMP_FORWARD_IF_FALSE 生成问题?可以将这些拆到一个独立的 PR

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个还没有,改回来了,昨晚没看清楚改错了

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

啊,我说 @gouzil 的 PR

@gouzil gouzil self-requested a review February 28, 2024 01:40
@diadestiny diadestiny dismissed stale reviews from gouzil and SigureMo via 00ac939 February 28, 2024 01:48
@gouzil gouzil merged commit 0651cba into PaddlePaddle:develop Feb 28, 2024
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants