已开启
[Requirement|需求建议]: 清理 AICPU 算子 G.CNS.03 成员函数 const 告警残留(5 处) #3272
liu-wei创建于  10 天前
liu-wei
liu-wei成员
10 天前 创建

背景

ops-math 仓 AICPU 算子仍存在 5 条 G.CNS.03 告警(不修改成员变量的成员函数应使用 const 修饰),为首批整改(#3217)及后续批次(#3268、PR #5881)合入后的无主残留项,属 #3264(整改 AICPU 算子代码质量告警)范围。本 issue 跟踪该批清理,对应 PR #5897。

告警清单(5 条 / 8 文件,+10/-10)

告警 位置 修复
G.CNS.03 strided_slice_v2_aicpu.h/.cpp CheckParam 补充 const(4 个被调成员均已为 const)
G.CNS.03 trans_data_aicpu.h/.cpp DealData 补充 const(依赖 BuildDealDataCtx/DealDataForHW 均为自由函数)
G.CNS.03 trans_data_aicpu.h/.cpp PaddingTwo 补充 const(依赖 GetPaddingTwoShape 已 const、AllocatePaddingBuffer 为自由函数)
G.CNS.03 real_div_aicpu.h/.cpp RealDivSameTypeCompute 补充 const(依赖 RealDivCompute 已 const)
G.CNS.03 mul_aicpu.h/.cpp MulSameTypeCompute 补充 const(依赖 MulCompute 已 const)

范围外项

  • trans_data 的 DispatchDealData G.CNS.04 ×3:维持 PR #5856 误报结论(aicpu::PtrToPtr<void, T> 无 const 重载,且 output_data_temp 确实被写入,不可 const);
  • strided_slice 的 G.EXP.27 ×4:由 #3267 另行跟进。

验证

  • 二进制级对比(父提交 vs 本提交,同编译参数):4 个改动 TU 的全部非符号节(共 4598 节)逐字节相同,仅 5 处成员函数符号修饰名 _ZN→_ZNK(含 _GLOBAL__sub_I 辅助符号)及符号表差异,未定义符号集合一致——零功能影响。

关联PR

https://gitcode.com/cann/ops-math/pull/5897

9.2.0 分支回移植

https://gitcode.com/cann/ops-math/pull/5903

likedislike
liu-weiliu-wei成员
10 天前 关联了pull request:[9.2.0] fix(aicpu): 补充5处成员函数const修饰(G.CNS.03残留)
liu-weiliu-wei成员
10 天前 修改了issue 的描述
陈思
陈思成员
10 天前 评论:

/assign @liu-wei

likedislike
CANN-robotCANN-robot成员
10 天前 将 liu-wei 设为负责人