已合并
fix: 修复BatchToSpaceND 等算子 codecheck 问题 #3501
zhanw_coding创建于 6月23日
fix: 修复BatchToSpaceND 等算子 codecheck 问题 #3501
已合并
zhanw_coding创建于 6月23日
zhanw_coding
zhanw_coding成员
6月23日

描述

fix: 修复BatchToSpaceND 等算子 codecheck 问题
master分支PR:

关联的Issue

https://gitcode.com/cann/ops-math/issues/1944

测试

文档更新

类型标签

  • Bug修复
  • 新特性
  • 性能优化
  • 文档更新
  • 其他,请描述:code check
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhanw_coding 的贡献)
zhanw_codingzhanw_coding成员
6月23日 创建了 pull request,commit b12fb73a
zhanw_codingzhanw_coding成员
6月23日 关联了issue:[Bug-Report|缺陷反馈]: BatchToSpaceND、Transpose算子等存在Codecheck问题
CANN-robotCANN-robot成员
6月23日 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
6月23日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
6月23日 评论:

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
*/*/op_host/*_def.cpp 王永光, 宋恺, wangrui_ (3/2) 王永光 (1/1)
conversion/broadcast_to 宋恺, wangrui_ (2/2) 宋恺, wangrui_ (2/1)
conversion/im2col 宋恺, wangrui_ (2/2) 宋恺, wangrui_ (2/1)
conversion/strided_slice 宋恺, wangrui_ (2/2) 宋恺, wangrui_ (2/1)
conversion/strided_slice_v3 宋恺, wangrui_ (2/2) 宋恺, wangrui_ (2/1)
conversion/trans_data wangrui_, 宋恺 (2/2) wangrui_, 宋恺 (2/1)
math/arg_max_with_value 宋恺, wangrui_ (2/2) wangrui_, 宋恺 (2/1)
repo-cann/ops-math wangrui_, 宋恺 (2/2) 宋恺, wangrui_ (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
zhanw_coding
zhanw_coding成员
6月23日 评论:

compile

likedislike
zhanw_codingzhanw_coding成员
6月23日 update merge request[project id: 7649531, iid: 3501, commit_id: 9076c65d8b552eab0ae8eedcc2c11c6b0d069dae] virtual merging success
CANN-robotCANN-robot成员
6月23日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
6月23日 评论:

流水线任务触发成功
任务链接 [2e811ed644fd42ec98f45e075a1f3ac1][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_A5 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_A5 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_harmony-infer ✅ SUCCESS >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
UT_Test ✅ SUCCESS
UT_Test_experimental ✅ SUCCESS
UT_Test_kernel ✅ SUCCESS
PreSmoke_A900 ✅ SUCCESS >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>

[2026-06-23 22:01:55]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6月23日 评论:

流水线任务触发成功
任务链接 [df425dc14e404607a639cf008a7cf013][流水线指导]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Pr ✅ SUCCESS
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_codestyle ⚠️ WARNING >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>

[2026-06-23 21:57:59]    CI执行结束

likedislike
CANN-robotCANN-robot成员
6月23日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
6月23日 添加了label:ci-pipeline-passed
zhanw_codingzhanw_coding成员
6月23日 修改了pull request 的描述
atomgit-bot
atomgit-bot
6月24日 评论:

变更摘要

本 PR 主要针对 BatchToSpaceNDBroadcastToIm2colMatrixSetDiagMemSetV2PadV3PadV3GradStridedSliceStridedSliceV2StridedSliceV3TransDataArgMaxWithValueSincTruncateDiv 等多个算子的 host tiling 与 kernel 实现进行 codecheck(静态代码检查)问题修复,涉及代码风格、类型安全、常量命名规范化等方面的调整。改动集中在 op_host tiling 逻辑和 op_kernel 核心计算实现文件中,以消除静态分析工具报告的违规项。

主要改动

  • truncate_div_apt.cpp 大规模重构truncate_div 算子的 kernel 入口文件进行了显著重构(+133/-57),重新组织了各数据类型的模板分发逻辑(truncate_div_halftruncate_div_bfloat16truncate_div_floattruncate_div_int8 等),新增了 int64_tint16_t 等数据类型的处理分支,并引入 canUseMul 模板参数优化标量取倒数乘法路径。

  • batch_to_space_nd 算子 codecheck 修复batch_to_space_nd_tiling_arch35.cpp 的 tiling 类成员和常量定义进行了规范化调整,batch_to_space_nd_large_c.hbatch_to_space_nd_small_c.h 中新增了多个 constexpr static 具名常量(如 AXIS_OFFSET_WAXIS_OFFSET_HBLOCK_AXIS_OFFSET_HMIN_SPATIAL_DIMS 等)以替代魔法数字,改善代码可读性。

  • broadcast_to tiling 常量与头文件规范化broadcast_to_tiling_base.hbroadcast_to_tiling_base.cpp 中的常量定义(如 brctoMaxDMADimNumbrctoMaxADimNumbrctoMaxBDimNum)改用 constexpr size_t 并通过 static_cast 消除类型窄化警告;同时调整了 GetHardwareInfo 模板方法的相关实现。

  • trans_data tiling 结构体类型修复transdata_tiling_arch35.htransdata_tiling_arch35.cpp 中对 TransDataCompileInfo 结构体成员类型、coreNum_ 等变量的类型进行了调整(如 uint32_t/int64_t 之间的统一),并对 GetHardwareInfo 中的平台参数校验逻辑进行了规范化。

  • matrix_set_diagstrided_slice_v2strided_slice_v3 等算子 tiling 代码调整:这些算子的 host tiling 文件中引入了 constexpr 常量替代部分字面量,调整了局部变量声明顺序与初始化方式,sinc_def.cpp 移除了一行冗余代码,以通过 codecheck 规则。

likedislike
atomgit-bot
atomgit-bot
6月24日 评论:

代码审查

✅ 未发现问题

likedislike
songkai111成员
6月24日 评论:

/lgtm
/approve

likedislike
RuiWang_成员
6月24日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
6月24日 添加了label:lgtm
wangyongguang成员
6月24日 评论:

/approve

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