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

[phi decoupling] move platform/transform.h to phi #50498

Merged
merged 8 commits into from
Feb 17, 2023

Conversation

huangjiyi
Copy link
Member

PR types

Others

PR changes

Others

Describe

  • move fluid/platform/tranform.h and fluid/platform/details/cuda_transform_iterator_cast.h to phi/common/

@paddle-bot
Copy link

paddle-bot bot commented Feb 14, 2023

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

@@ -23,7 +23,7 @@ limitations under the License. */
#include "thrust/device_ptr.h"

namespace paddle {
namespace platform {
namespace experimental {
Copy link
Contributor

Choose a reason for hiding this comment

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

把命名空间直接改成phi吧,paddle::experimental是历史遗留问题,后边要废弃,另外把details目录删了吧,这个目录好像只有这一个文件,不需要给他增加目录层级

Copy link
Member Author

@huangjiyi huangjiyi Feb 15, 2023

Choose a reason for hiding this comment

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

该文件的内容已合入 transform.h

#endif

namespace paddle {
namespace platform {
namespace experimental {
Copy link
Contributor

Choose a reason for hiding this comment

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

同上,命名空间的修改

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

YuanRisheng
YuanRisheng previously approved these changes Feb 16, 2023
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

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

@YuanRisheng YuanRisheng merged commit fe33279 into PaddlePaddle:develop Feb 17, 2023
@huangjiyi
Copy link
Member Author

@huangjiyi huangjiyi changed the title [phi decoupling] move platform/transform to phi [phi decoupling] move platform/transform.h to phi Feb 20, 2023
@huangjiyi
Copy link
Member Author

@YuanRisheng

@huangjiyi huangjiyi deleted the decouple_transform branch March 8, 2023 15:54
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.

4 participants