已合并
修改相关issue文档 #5310
jcmrn0930创建于 16 天前
修改相关issue文档 #5310
已合并
jcmrn0930创建于 16 天前
已删除 :issue合入到cann/asc-devkitmaster
jcmrn0930成员
16 天前

描述

GetResGroupBarrierWorkSpaceSize 返回类型为 uint32_t,但在错误路径返回 -1。-1 隐式转换为 uint32_t 值 4294967295,据此修改相关用户文档资料

关联的Issue

关联Issue #1543
关联Issue #1544
关联Issue #1545

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 jcmrn0930 的贡献)
Jjcmrn0930成员
16 天前 创建了 pull request,commit 8322f27d
Jjcmrn0930成员
16 天前 关联了issue:【缺陷报告】数据类型截断 - 文件platform_ascendc.cpp - 函数GetLibApiWorkSpaceSize - 行号271,【缺陷报告】数据类型截断 - 文件platform_ascendc.cpp - 函数GetResCubeGroupWorkSpaceSize - 行号287,【缺陷报告】数据类型截断 - 文件platform_ascendc.cpp - 函数GetResGroupBarrierWorkSpaceSize - 行号298
atomgit-bot
atomgit-bot
16 天前 评论:

变更摘要

本 PR 针对 GetResGroupBarrierWorkSpaceSize 返回类型为 uint32_t、但在错误路径返回 -1(隐式转换为 4294967295)的问题,据此修改相关用户文档资料;同时,在 InitGlobalMemoryImplinit_global_memory_v200_impl.hinit_global_memory_v220_impl.h)中新增对 popSize 大于 0 的断言校验,避免其取值为 0 时后续 size / popSize 计算出现除零问题。

主要改动

  • v200 实现新增 popSize 非零断言: 在 init_global_memory_v200_impl.hInitGlobalMemoryImpl 中,于计算 roundtail 之前新增 ASCENDC_ASSERT((popSize > 0), ...),确保 popSize 大于 0 后再进行除法运算。
  • v220 实现同步新增同一断言: 在 init_global_memory_v220_impl.hInitGlobalMemoryImpl 中同样新增 ASCENDC_ASSERT((popSize > 0), ...),使 v200 与 v220 两个版本在该处的校验行为保持一致。
likedislike
不准确?
atomgit-bot
atomgit-bot
16 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
16 天前 添加了label:stat/needs-squash
此处折叠了66条消息 查看更多
CANN-robotCANN-robot成员
16 天前 添加了label:approved
CANN-robot
CANN-robot成员
16 天前 评论:

The MR can not be merged, because of CodeReview discussion not resolved

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
Jjcmrn0930成员
16 天前 解决了最后一个问题
CANN-robotCANN-robot成员
16 天前 关闭了关联的issue
CANN-robotCANN-robot成员
16 天前 合入了pull request