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

[OpenCL][kernel]refactoring_transpose_kernel test=develop #8341

Merged

Conversation

zhenlin-work
Copy link
Collaborator

重构了transpose kernel层cc代码,并支持了2 3 4维所有非本身的permutation组合的OpenCL实现。

@paddle-bot-old
Copy link

Thanks for your contribution!

@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Jan 24, 2022
@PaddlePaddle PaddlePaddle unlocked this conversation Jan 24, 2022
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Jan 24, 2022
@PaddlePaddle PaddlePaddle unlocked this conversation Jan 24, 2022
daming5432
daming5432 previously approved these changes Jan 24, 2022
Copy link
Collaborator

@daming5432 daming5432 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
Collaborator

@zhaoyang-star zhaoyang-star left a comment

Choose a reason for hiding this comment

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

也将 lite/kernels/opencl/transpose_image_compute.cc 中的 Line 353-355 删除吧。

zhaoyang-star
zhaoyang-star previously approved these changes Jan 25, 2022
Copy link
Collaborator

@zhaoyang-star zhaoyang-star 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
Collaborator

@sprouteer sprouteer left a comment

Choose a reason for hiding this comment

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

LGTM

@zhenlin-work zhenlin-work merged commit 5cd9b89 into PaddlePaddle:develop Feb 8, 2022
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