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

Update relu_arm.cpp #189

Merged
merged 3 commits into from
Nov 14, 2017
Merged

Update relu_arm.cpp #189

merged 3 commits into from
Nov 14, 2017

Conversation

Feng137
Copy link
Contributor

@Feng137 Feng137 commented Nov 13, 2017

relu的修正!感觉少了个感叹号!

relu的修正!prelu也有相似的问题。
@nihui
Copy link
Member

nihui commented Nov 13, 2017

relu_arm slope != 0.f 确实少了感叹号,inplace 几个没有少,vld1 和 vst1 是同一块地址
还有就是能不能也帮改下 prelu_arm 少了感叹号

@Feng137
Copy link
Contributor Author

Feng137 commented Nov 14, 2017

嗯 对 inplace不用加感叹号 我加多了。好的,我把 prelu_arm 也修改以下重新提交。

@nihui
Copy link
Member

nihui commented Nov 14, 2017

谢谢!!

@nihui nihui merged commit f232c1a into Tencent:master Nov 14, 2017
ghimiredhikura pushed a commit to ghimiredhikura/NCNNSFD that referenced this pull request Jul 26, 2020
* Update relu_arm.cpp

* Update prelu_arm.cpp
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