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

Remove LoDTensor using in fluid (Part 1) #46663

Merged
merged 4 commits into from
Oct 11, 2022

Conversation

chenwhql
Copy link
Contributor

@chenwhql chenwhql commented Sep 30, 2022

PR types

Function optimization

PR changes

Others

Describe

Remove LoDTensor using in fluid (Part 1)

移除框架内 paddle::framework::LoDTensor 和 framework::LoDTensor 的使用,使基础概念更清晰一致,为后续移除 lod_tensor.h 中的 using LoDTensor = phi::DenseTensor 做准备

前序PR及背景见:#46432

YuanRisheng
YuanRisheng previously approved these changes Oct 11, 2022
zyfncg
zyfncg previously approved these changes Oct 11, 2022
ZzSean
ZzSean previously approved these changes Oct 11, 2022
Copy link
Contributor

@ZzSean ZzSean left a comment

Choose a reason for hiding this comment

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

LGTM for CI-OP-Benchmark

@jiahy0825 jiahy0825 self-requested a review October 11, 2022 02:21
jiahy0825
jiahy0825 previously approved these changes Oct 11, 2022
Copy link
Contributor

@jiahy0825 jiahy0825 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

7 participants