fix(cleancode): AICPU算子告警整改（第三批）-ops-nn-AtomGit

已合并

fix(cleancode): AICPU算子告警整改（第三批） #10997

liu-wei创建于 22 天前从liu-wei/ops-nn： fix/warning-cleanup-v3 合入到 cann/ops-nn： master

fix(cleancode): AICPU算子告警整改（第三批） #10997

已合并

liu-wei创建于 22 天前

从liu-wei/ops-nn fix/warning-cleanup-v3合入到cann/ops-nn master

liu-wei成员

22 天前

描述

ops-nn AICPU 算子告警整改第三批，修复 18 条告警（G.CNS.03 ×12 / G.EXP.27+36 ×5 / G.CNS.04 ×1），共 14 个文件 +29/-29：

告警	算子	修复
G.CNS.03 ×12	log_softmax_v2、scatter_elements、sparse_segment_mean、sparse_segment_sum、sparse_to_dense、tensor_scatter_update	12 处成员函数补充 `const`（声明与定义同步，被调成员均为 const 或自由函数）
G.EXP.27/36 ×4	softmax_v2	`NormalCheck(...)`/`detail::SoftmaxV2Check(...)` 两处 `?:` 条件改为显式 `!= KERNEL_STATUS_OK`
G.EXP.27 ×1	reverse_sequence	`reverseNum % kEven` 显式比较 `!= 0`
G.CNS.04 ×1	tensor_scatter_update	`GetBatchStrides` 的 `outer_shape` 参数加 `const`（仅读取）

不处理项（附判定依据）

浮点 == static_cast<T>(0) ×2（log_softmax_v2/softmax_v2 的除零防护）：保持语义，建议工具豁免；
sparse_to_dense G.STD.02 ×4：char* 为字节缓冲非字符串，误报；
EigenSparseToDense 的 SparseTensor& st：st.ToDense<>() 受 Eigen API 限制，疑似误报。

验证

8 个改动 TU 语法级编译通过（g++ -std=c++14）；
二进制级对比（父提交 467b22d6c vs 本提交 7f12ebcbc，同编译参数 g++ -O2 -std=c++14）：8 个 TU 全部机器码/数据节内容多重集逐字节一致（合计 1298 节），仅 const 成员函数 mangled 名（符号表及 scatter_elements 15 节、sparse_to_dense 1 节的内嵌签名串 ZN→ZNK）变化——零功能影响；
CI 流水线待跑。

关联的Issue

https://gitcode.com/cann/ops-nn/issues/6114

类型标签

♻️ 重构
🧹 代码清理

Pull Request已成功合入, 合并人@CANN-robot

（感谢 liu-wei 的贡献）

liu-wei成员

22 天前创建了 pull request，commit 31d8d5ae

atomgit-bot

22 天前评论：

变更摘要

本 PR 是 ops-nn AICPU 算子告警整改第三批，针对多个算子的静态检查告警进行代码清理，改动以成员函数补充 const、条件表达式显式化为主，不改变运行逻辑。整改覆盖 log_softmax_v2、scatter_elements、sparse_segment_mean、sparse_segment_sum、sparse_to_dense、tensor_scatter_update、softmax_v2、reverse_sequence 等算子的 AICPU 实现，声明与定义同步修改。

主要改动

成员函数补充 const 限定（G.CNS.03）: 为 LogSoftmaxV2Check、LogSoftmaxV2Compute、ScatterElementsCpuKernel::UpdateOutput/DispatchByDataType/DoCompute、SparseSegmentMeanCpuKernel::ComputeKernel/ComputeKernelWithType、SparseSegmentSumCpuKernel 同名方法、SparseToDenseCpuKernel::ValidParam/ParallelSetDefaultValue、TensorScatterUpdateCpukernel::GetInputAndCheck 等在头文件声明与 .cpp 定义中同步追加 const，标识其不修改对象状态。
条件表达式显式化（G.EXP.27/36）: softmax_v2_aicpu.cpp 中将 SoftmaxV2Check 与 SoftmaxV2CpuKernel::Compute 里以 NormalCheck(...)、detail::SoftmaxV2Check(...) 作为 ?: 判断条件的写法改为显式 != KERNEL_STATUS_OK 比较，语义保持一致。
取模判断显式化（G.EXP.27）: reverse_sequence_aicpu.cpp 中将 reverseNum % kEven 改为 (reverseNum % kEven) != 0，使布尔判断意图更明确。
只读指针参数补充 const（G.CNS.04）: tensor_scatter_update_aicpu.cpp 中 GetBatchStrides 的 outer_shape 参数由 uint64_t* 改为 const uint64_t*，标明该参数仅被读取。