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][Buffer]Add reshape buffer opencl #9193

Merged
merged 4 commits into from
Aug 11, 2022

Conversation

sprouteer
Copy link
Collaborator

  1. 为了解决image2d版本,需要对内存重排导致的性能问题
  2. 为了解决部分模型在adreno gpu上超出image2d范围报错问题

Copy link
Contributor

@liyancas liyancas left a comment

Choose a reason for hiding this comment

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

LGTM

@liyancas liyancas removed the request for review from chenjiaoAngel July 21, 2022 07:26
// See the License for the specific language governing permissions and
// limitations under the License.

#include <gtest/gtest.h>
Copy link
Collaborator

Choose a reason for hiding this comment

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

opencl 的单测能复用lite/tests/kernels下面的吗?

hong19860320
hong19860320 previously approved these changes Jul 21, 2022
Copy link
Collaborator

@hong19860320 hong19860320 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

@zhupengyang zhupengyang left a comment

Choose a reason for hiding this comment

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

LGTM

@sprouteer sprouteer merged commit 924a432 into PaddlePaddle:develop Aug 11, 2022
QShiX pushed a commit to QShiX/Paddle-Lite that referenced this pull request Aug 13, 2022
* add reshape buffer opencl test=develop

* fix fc image2d bug test=develop

* fix fc image2d bug test=develop

* fix opencl reshape op_test 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.

4 participants