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] Fix invalid arg size in instance_norm #6064

Merged
merged 1 commit into from
May 12, 2021

Conversation

zhaoyang-star
Copy link
Collaborator

setArg的参数类型需与kernel内的参数类型一致,如果不一致,不同OpenCL编译器可能无法完成隐式类型转换导致如下的运行时错误。
image

@paddle-bot-old
Copy link

Thanks for your contribution!

@zhaoyang-star zhaoyang-star requested a review from daming5432 May 12, 2021 02:33
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

@zhaoyang-star zhaoyang-star merged commit 38c50bb into PaddlePaddle:develop May 12, 2021
@zhaoyang-star zhaoyang-star deleted the fix_in_norm branch May 12, 2021 08:59
zhaoyang-star added a commit to zhaoyang-star/Paddle-Lite that referenced this pull request Jun 13, 2021
zhaoyang-star added a commit to zhaoyang-star/Paddle-Lite that referenced this pull request Jun 15, 2021
daming5432 pushed a commit that referenced this pull request Jun 16, 2021
* [OpenCL] Fix select fp32 compile crash (#6006)

* [Pass] Add opencl_kernel_place_correct_pass (#6037)

* [OpenCL] Fix invalid arg size in instance_norm (#6064)

* [OpenCL][Kernel] Add concat multi inputs kernel except channel is not aligned (#6075)

* [OpenCL][Bugfix] Fix target choose in opencl_kernel_place_correct_pass  (#6079)

* [OpenCL] fix kernel select of concat (#6158)

* [OpenCL] BindTargets KOpenCL for conv_conv_fuse_pass (#6125)

* test=develop

* [UTest] Loose abs_error for group_norm and instance_norm (#6188)

* loose group_norm abs_err. test=develop
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.

2 participants