Skip to content

[Dy2St] Add FLAGS_specialize_device_in_dy2st to set PlaceAttribute for DataOp based on input tensor's place before lowering#73405

Merged
SigureMo merged 2 commits into
PaddlePaddle:developfrom
0x3878f:sot-support-7
Jun 18, 2025
Merged

[Dy2St] Add FLAGS_specialize_device_in_dy2st to set PlaceAttribute for DataOp based on input tensor's place before lowering#73405
SigureMo merged 2 commits into
PaddlePaddle:developfrom
0x3878f:sot-support-7

Conversation

@0x3878f

@0x3878f 0x3878f commented Jun 17, 2025

Copy link
Copy Markdown
Contributor

PR Category

Execute Infrastructure

PR Types

New features

Description

Add FLAGS_specialize_device_in_dy2st, default value is False. It will set PlaceAttribute for DataOp based on input tensor's place before lowering when FLAGS_specialize_device_in_dy2st=1

NOTE:
Performance may decrease when a CPU Tensor is copied to a device multiple times.
Consider applying CSE to solve this problem.

pcard-67164

@paddle-bot

paddle-bot Bot commented Jun 17, 2025

Copy link
Copy Markdown

你的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.

@0x3878f 0x3878f requested a review from SigureMo June 17, 2025 09:48
SigureMo
SigureMo previously approved these changes Jun 17, 2025

@SigureMo SigureMo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTMeow 🐾

@SigureMo SigureMo merged commit 5690709 into PaddlePaddle:develop Jun 18, 2025
48 of 51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants