[Doc][Polish] avoid memory leak and explain some points #54
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
muyuuuu
commented
Nov 25, 2024
muyuuuu
commented
- 避免内存泄漏
- 一次 IO，多次计算那里是我个人的理解，在之前 opencl 优化算子中也用到了，算一种通用思路吧
- 矩阵大小不为 32 的倍数时会有段错误。我见过的优化方法是先取出能整除 32 的图像区域进行 cuda 加速，边界部分用 C 处理。我不确定百度的优化方法，就没写解决方案
- TM 的解释，我看了好久看懂了 TM 的用法，擅自主张加了个解释
|
感谢对文档的补充！ “矩阵大小不为 32 的倍数时会有段错误” 这个问题其实在很多sgemm优化算法里面都会遇到，文档里面的文章主要还是学习这些优化方法，考虑特别多边界情况的话会让代码变的非常复杂 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.