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

Persuading more efficient memory format to be preferred #44078

Merged
merged 4 commits into from
Jul 5, 2022

Conversation

jczaja
Copy link
Contributor

@jczaja jczaja commented Jul 4, 2022

PR types

Performance optimization

PR changes

OPs

Describe

This PR improves preferences of more efficient memory formats when guessing memory format out of memory descriptor
It does speedup mobilenetv1 int8 batch_size 50 scenario

@jczaja jczaja added the Intel label Jul 4, 2022
@jakpiase jakpiase self-requested a review July 5, 2022 01:12
Copy link
Contributor

@jakpiase jakpiase left a comment

Choose a reason for hiding this comment

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

LGTM

@jczaja jczaja requested review from lidanqing-intel and wozna July 5, 2022 07:18
Copy link
Contributor

@lidanqing-intel lidanqing-intel left a comment

Choose a reason for hiding this comment

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

LGTM

@paddle-bot-old
Copy link

paddle-bot-old bot commented Jul 5, 2022

很抱歉,经过我们的反复讨论,你的PR暂未达到合入标准,请阅读飞桨原生算子开发规范,你可以重新提交新的PR,我们先将此PR关闭,感谢你的贡献。
Sorry to inform you that through our discussion, your PR fails to meet the merging standard (Reference: Paddle Custom Operator Design Doc). You can also submit an new one. Thank you.

@jczaja
Copy link
Contributor Author

jczaja commented Jul 5, 2022

@jiangjiajun Please start your review

@paddle-bot-old
Copy link

paddle-bot-old bot commented Jul 5, 2022

你的PR已合入Paddle库,请关注后续测试结果。
Your PR has been merged into the repository. An official integration test will be conducted later. Stay tuned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants