已关闭
fatrelu_mul 算子 codecheck 检视意见整改 #6286
wwwzzzxxx创建于  21 天前关闭于  20 天前
wwwzzzxxx
wwwzzzxxx
21 天前 创建

背景

对 ops-nn 仓库 activation/fatrelu_mul 算子进行 codecheck 静态检视(allrule-c 与 c++ 规则集),扫描出若干规范性问题,需要整改。

问题清单(4 个文件,共 37 条)

  1. 魔法数字(G.CNS.02):kernel 中 blockCount 上限 4095、TBuf 槽位下标 0~5;host tiling 中 dtype 字节数 4/2、末维折半因子 2、buffer 份数 3。
  2. 注释掉的死代码(G.OTH.01 / G.EXP.43):kernel 头文件第 29 行注释残留分号,apt.cpp 中 [REF_SAMPLE] 注释块。
  3. 空格式函数头注释(G.CMT.04):tiling 文件 10 处单独成行的空 // 注释。
  4. 指针参数未加 const(G.CNS.04):FatreluMulCheckInput / FatreluMulFillAndLogTilingData 的 ctx 参数。
  5. 有符号/无符号类型转换回绕隐患(G.EXP.21):infershape 中 SIZE_2 为 size_t,导致 lastDim 参与无符号运算。
  6. 位运算操作数类型不当(G.EXP.27):lastDim & 1 判断奇偶,应改为取模。

修复方式

以具名常量替换魔法数字、删除死代码与空注释、补充 const、统一有符号类型,不改变算子计算逻辑。

关联 PR

https://gitcode.com/cann/ops-nn/merge_requests/11343

likedislike
wwwzzzxxxwwwzzzxxx
21 天前 关联了pull request:fix(fatrelu_mul): 修复codecheck检视意见(魔法数字、注释死代码、空函数头注释、const参数、有符号类型转换)
wwwzzzxxxwwwzzzxxx
21 天前 修改了issue 的描述
yuning_chenyuning_chen成员
21 天前 将 m0_71149169 设为负责人
CANN-robotCANN-robot成员
20 天前 关闭了 issue
CANN-robotCANN-robot成员
20 天前 添加了label:resolved