已合并
aclnnSplitTensor增加输出数据类型一致的校验,及样例指针重复释放修复 #3773
sunhao_hw创建于 7月3日
aclnnSplitTensor增加输出数据类型一致的校验,及样例指针重复释放修复 #3773
已合并
sunhao_hw创建于 7月3日
sunhao_hw成员
7月3日

描述

  • 在aclnnSplitTensor的CheckTypeValid里增加输出子块的数据类型一致的校验,当输出个数大于等于2时触发校验
  • 发现aclnnSplitTensor和aclnnSplitWithSize的demo中存在指针重复释放的问题,进行修复
  • 使用clang-format整改代码风格

关联的Issue

测试

构造输出类型不一致的输出子块List,自验拦截生效

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 sunhao_hw 的贡献)
Ssunhao_hw成员
7月3日 创建了 pull request,commit d3eee40c
CANN-robotCANN-robot成员
7月3日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7月3日 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
*/*/docs/acl*.md 吴瑶, 周奇龙 (2/2) 吴瑶 (1/1)
conversion/split_v 唐燕峰, 周奇龙 (2/2) 唐燕峰, 周奇龙 (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

sunhao_hw, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
atomgit-bot
atomgit-bot
7月3日 评论:

变更摘要

本次 PR 在 aclnnSplitTensorCheckDtypeValid 函数中增加了输出子块数据类型一致性校验,同时重构了原有的数据类型检查控制流。当 aclTensorList 的输出个数大于等于 2 时,会逐个比较各输出 Tensor 与首个输出 Tensor 的数据类型是否一致,不一致则通过 OP_CHECK_DTYPE_NOT_SAME 宏拦截。此外,原有的 if (out->Size() > 0) 分支在检查完 DTYPE_SUPPORT_LIST_950 后直接 return true,跳过了对 self 及其他输出的 DTYPE_SUPPORT_LIST 检查,此次一并修复为 if-else 结构,确保两条分支各自完整覆盖 self 和所有输出 Tensor 的类型校验。

主要改动

  • 新增输出数据类型一致性校验:在 CheckDtypeValid 末尾增加判断 out->Size() >= 2 时,遍历所有输出 Tensor 并使用 OP_CHECK_DTYPE_NOT_SAME 与第一个输出对比,确保所有子块类型一致。
  • 重构数据类型检查控制流:将原来 if (out->Size() > 0) 分支中提前 return true 的逻辑改为 if-else 结构,使 DTYPE_SUPPORT_LIST_950 分支与 DTYPE_SUPPORT_LIST 分支各自完整地对 self 和每个输出 Tensor 执行 OP_CHECK_DTYPE_NOT_SUPPORT 检查,消除原控制流中遗漏校验的问题。
likedislike
atomgit-bot
atomgit-bot
7月3日 评论:

代码审查

✅ 未发现问题

likedislike
此处折叠了62条消息 查看更多
FelixTang7成员
7月6日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
7月6日 添加了label:approved
sunday成员
7月6日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
7月6日 添加了label:lgtm
CANN-robotCANN-robot成员
7月6日 合入了pull request