master

cann-robotfix: 修复 cleancode 告警

c1d937ae创建于 3 天前历史提交

文件	最后提交记录	最后更新时间
arch22	修复matmulalltoall量化场景bias非bf时精度出错的问题 Co-authored-by: SimpleBright_Man<tangpanrui1@huawei.com> # message auto-generated for no-merge-commit merge: !5246 merge fixMMAAkernelEntry into master 修复matmulalltoall量化场景bias非bf时精度出错的问题 Created-by: SimpleBright_Man Commit-by: SimpleBright_Man Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 修改tilingkey，删除了quant_bf16，新增了核函数入口处的分支 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2373 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!5246	20 天前
arch35	[AICPU] MatmulAlltoAll算子根据环境变量选择commMode通信引擎 Co-authored-by: adamlwang<wangjingquan2@huawei.com> # message auto-generated for no-merge-commit merge: !5182 merge aicpu into master [AICPU] MatmulAlltoAll算子根据环境变量选择commMode通信引擎 Created-by: adamlwang Commit-by: adamlwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> [AICPU] 算子根据环境变量选择commMode通信引擎 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2383 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [x] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # 代码检视报告 **项目名称**：matmul_allto_all 算子 PR #5182 检视报告 **检视范围**：PR diff（8 个变更文件） **检视人**：Turing Team **检视日期**：2026-05-12 **检视模式**：PR 检视模式（重点：TOPK 高频问题 + 红线问题） --- ## 🔍 检视概览 | 统计项 | 数值 | | ---- | ---- | | 检视条款总数 | 8 条 | | 通过条款数 | 8 条 | | 发现问题数 | 0 个 | | 需关注点 | 1 个（编译影响） | **核心结论**：变更代码符合 TOPK 高频问题和红线问题要求，无高风险问题。建议关注模板参数组合数量翻倍对编译时间和二进制体积的影响。 --- ## ✅ 检视条款通过情况 ### TOPK 高频问题（4 条） | 条款ID | 条款名称 | 检视结果 | 说明 | | ------ | -------- | -------- | ---- | | TOPK-1 | 必须校验函数返回值 | ✅ PASS | GetCommModeFromEnv() 设计保证返回有效值 | | TOPK-3 | 生命周期内使用局部变量指针 | ✅ PASS | 变更代码无野指针风险 | | TOPK-8 | GM 内存偏移必须用 int64 | ✅ PASS | 变更代码未涉及 GM 偏移计算 | | TOPK-10 | 可整数计算时不允许转浮点 | ✅ PASS | 变更代码使用整数类型运算 | ### C++ 安全编码红线（4 条） | 条款ID | 条款名称 | 检视结果 | 说明 | | ------ | -------- | -------- | ---- | | CPP-2.1 | 有符号整数运算不溢出 | ✅ PASS | 变更代码未涉及溢出风险 | | CPP-2.2 | 无符号整数运算不回绕 | ✅ PASS | 变更代码无回绕风险 | | CPP-3.5 | 指针使用前判空 | ✅ PASS | 变更代码无新增指针使用 | | API-12 | CrossCoreSetFlag/WaitFlag 配对 | ✅ PASS | 变更代码无核间同步逻辑 | --- ## 🔶 需关注点（非红线问题） ### 关注点ID：NOTE-001 | 级别：MEDIUM（需关注） **代码路径**：mc2/matmul_allto_all/op_kernel/arch35/matmul_allto_all_tiling_key.h **问题描述**：模板参数组合数量从 12 个扩展为 24 个（新增 COMMMODE 参数），可能导致： 1. 编译时间显著增加（模板实例化数量翻倍） 2. 二进制体积显著增大（每个组合独立编译） **影响范围**： - 模板参数：新增 COMMMODE（1 位宽度，取值 0 或 1） - 组合数量：12 × 2 = 24 个（覆盖完整，无遗漏） **建议**： - 确认编译时间和二进制体积是否在可接受范围内 - 如影响较大，可考虑优化方案（如动态分支替代静态模板） --- ## 📋 变更文件清单 | 文件类型 | 文件路径 | 代码侧别 | 变更类型 | | -------- | -------- | -------- | -------- | | op_api | aclnn_matmul_allto_all.cpp | Tiling侧 | 新增环境变量读取逻辑 | | op_api | aclnn_quant_matmul_allto_all.cpp | Tiling侧 | 新增环境变量读取逻辑 | | op_graph | matmul_allto_all_gen_task.cpp | Tiling侧 | 新增通信模式分支判断 | | op_host | fp_matmul_allto_all_tiling_base.cpp | Tiling侧 | 新增通信模式 Tiling 参数 | | op_host | kc_quant_matmul_allto_all_tiling_base.cpp | Tiling侧 | 新增通信模式 Tiling 参数 | | op_host | mx_quant_matmul_allto_all_tiling_base.cpp | Tiling侧 | 新增通信模式 Tiling 参数 | | op_kernel | matmul_allto_all_tiling_key.h | Kernel侧 | 新增 COMMMODE 模板参数 | | op_kernel | matmul_allto_all_apt.cpp | Kernel侧 | 新增通信服务器类型选择逻辑 | --- ## 📊 检视统计 **检视条款覆盖**： - TOPK 高频问题：4/13 条（重点条款） - C++ 安全编码红线：4/32 条（重点条款） - API 红线问题：1/9 条（重点条款） **变更代码行数**： - 新增代码：约 60 行 - 修改代码：约 100 行 - 删除代码：约 30 行 --- ## 报告生成时间 2026-05-12 18:00:00 ## 报告状态 已完成检视，无高风险问题，建议关注编译影响 See merge request: cann/ops-transformer!5182	20 天前
common	fix: 修复 cleancode 告警 Co-authored-by: libohao6<libohao3@huawei.com> # message auto-generated for no-merge-commit merge: !5954 merge fix/cleancode-master into master fix: 修复 cleancode 告警 Created-by: libohao6 Commit-by: libohao6 Merged-by: cann-robot Description: ## 描述 修复基础算子主线cleancode问题。 ## 关联的Issue [Bug-Report|缺陷反馈]: MC2基础算子主线存在cleancode问题 #2676 ## 测试 二级冒烟。 ## 文档更新 不涉及。 ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR #5489 代码检视报告 **PR**: https://gitcode.com/cann/ops-transformer/pull/5489 **标题**: merge 'fix-cleancode-issues' into 'master' **作者**: libohao6 | **范围**: 19 files, +453 -531 | **日期**: 2026-05-21 --- ## 改动分类 | 类型 | 改动数 | 说明 | |------|-------|------| | 删除未使用函数 | 5个 | matmul_util.cpp删3个static函数 + aclnn_matmul_reduce_scatter_v2.cpp删CreateWinTensor | | 删除冗余变量 | 2个 | x1Dtype/biasDtype 在 aclnn_weight_quant_matmul_all_reduce.cpp | | const修饰添加 | 5处 | 多个Tiling函数context参数加const | | 位操作类型修复 | 1处 | int32_t code → uint32_t bitCode | | 魔法数字→常量 | 1处 | alignas(8) → alignas(L2_CACHE_ALIGNMENT) | | 变量重命名 | 多处 | Kernel侧blockLocCoord→blockLocCoord_等; Host侧index→idx/i | | 参数名拼写修正 | 1处 | rcfCfg→rcsCfg | | include guard | 1处 | matmul_reduce_scatter_v2_tiling_common.h | | 头文件清理 | 3处 | 移除冗余#include | | 未使用参数标记 | 5处 | (void)opType等 | | 注释风格更新 | 大量 | 决策树层级注释从======改为差异化标记 | | 代码格式修复 | 多处 | 空格/缩进/换行 | --- ## 假设检验检视结果 ### 1. 删除未使用函数 ✅ PASS **涉及文件**: matmul_util.cpp, aclnn_matmul_reduce_scatter_v2.cpp 删除5个static函数（CheckShapeValidWithTrans、ProcessEmptyTensor、ProcessEmptyTensorWithTrans、GetMatmulOpInfoWithTrans、CreateWinTensor）。均为static，无跨文件引用风险。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 5.2 资源泄露防护 | 删除是否遗漏调用点 | ✅ static函数仅文件内可见 | ### 2. const修饰添加 ✅ PASS **涉及文件**: all_gather_matmul_tiling_base.cpp/.h, quant_all_reduce_tiling.cpp, quant_reduce_scatter_tiling.cpp gert::TilingContext *context → const gert::TilingContext *context。GetNodeName()、GetAttrPointer()等方法为const兼容，头文件声明同步更新。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 1.1 静态类型安全 | const修饰是否导致编译错误 | ✅ 方法兼容const | ### 3. 位操作类型修复 ✅ PASS **涉及文件**: matmul_reduce_scatter_v2_aiv_mode_tiling.cpp int32_t code → uint32_t bitCode。对有符号数做右移(>>=)存在符号扩展问题，改为uint32_t消除风险。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 2.1 有符号整数不溢出 | int32_t右移符号扩展 | ✅ 修复正确 | | cpp-secure 2.2 无符号整数不回绕 | uint32_t位操作安全 | ✅ | ### 4. 魔法数字→常量 ✅ PASS **涉及文件**: unquant_matmul_all_reduce_tiling_data.h static constexpr size_t L2_CACHE_ALIGNMENT = 8; 替换 alignas(8)。值等价，不影响ABI。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 10.4 结构体字段末尾添加 | 常量定义位置 | ✅ 不影响ABI | ### 5. 冗余变量删除 ✅ PASS **涉及文件**: aclnn_weight_quant_matmul_all_reduce.cpp 删除 const auto x1Dtype = x1->GetDataType(); const auto biasDtype = bias->GetDataType();。两变量后续无引用，OP_CHECK_DTYPE_NOT_SAME宏自行获取dtype。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 3.1 禁止未初始化变量 | 删除后功能等价 | ✅ | ### 6. 参数名拼写修正 ✅ PASS **涉及文件**: matmul_reduce_scatter_tiling_base.cpp rcfCfg → rcsCfg（RCS=ReduceScatter拼写修正）。所有引用已同步替换。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | MC2 MC2-13 CCU通信数据量限制 | AdjustHCCLLimit逻辑是否保留 | ✅ 256MB限制保留 | ### 7. include guard添加 ✅ PASS **涉及文件**: matmul_reduce_scatter_v2_tiling_common.h 新增 #ifndef MATMUL_REDUCE_SCATTER_V2_TILING_COMMON_H 保护。命名与文件路径一致。 ### 8. 未使用参数标记 ✅ PASS **涉及文件**: quant_reduce_scatter_util_tiling.cpp 添加 (void)opType; (void)context; (void)xDimNum; (void)outputDim; 消除编译器警告。注释声明"Reserved for future extension"。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | TOPK TOPK-7 外部输入校验 | 校验逻辑是否削弱 | ✅ 未削弱 | ### 9. LOG参数安全 ✅ PASS **涉及文件**: allto_allv_quant_grouped_mat_mul_tiling_base.cpp 循环变量 index → idx/i。sendCounts[%lu] should be in [0, %lu], but get %lu — 3个%lu对应3个uint64_t(idx, bsk_, sendCounts[idx])，类型和数量匹配。 | 规范条款 | 检查项 | 结果 | |---------|--------|------| | cpp-secure 11.2 参数数量与占位符匹配 | 3个%lu=3个uint64_t | ✅ | | cpp-secure 11.3 参数类型与格式化说明符匹配 | uint64_t用%lu | ✅ | | MC2 MC2-07 专家参数合法性校验 | sendCounts/recvCounts校验完整 | ✅ | ### 10. 头文件清理 ✅ PASS **涉及文件**: matmul_reduce_scatter_tiling_base.h, quant_reduce_scatter_util_tiling.h, all_gather_matmul_tiling_base.h 移除冗余 #include "ops_utils.h" 和 #include "mc2_log.h"。 --- ## 规范条款引用汇总 | 规范来源 | 条款编号 | 条款名称 | 检查结果 | |---------|---------|---------|---------| | cpp-secure.md | 1.1 | 保证静态类型安全 | ✅ | | cpp-secure.md | 2.1 | 有符号整数运算不溢出 | ✅ | | cpp-secure.md | 2.2 | 无符号整数运算不回绕 | ✅ | | cpp-secure.md | 3.1 | 禁止使用未初始化的变量 | ✅ | | cpp-secure.md | 5.2 | 资源泄露防护 | ✅ | | cpp-secure.md | 10.4 | 结构体字段末尾添加 | ✅ | | cpp-secure.md | 11.2 | LOG参数数量与占位符必须匹配 | ✅ | | cpp-secure.md | 11.3 | LOG参数类型与格式化说明符必须匹配 | ✅ | | mc2-specific.md | MC2-07 | 专家参数合法性校验 | ✅ | | mc2-specific.md | MC2-13 | CCU通信数据量限制 | ✅ | | ascendc-topk.md | TOPK-1 | 必须校验函数返回值 | ✅ | | ascendc-topk.md | TOPK-7 | 融合规则/InferShape/Tiling外部输入校验 | ✅ | --- ## 总体结论 **✅ 无问题** — PR #5489 为代码清理类改动，所有10项检视均PASS。改动不引入安全风险，不削弱校验逻辑，规范合规性100%。 See merge request: cann/ops-transformer!5954	3 天前
matmul_allto_all_tiling.cpp	(register)replace socversion with npuArch Co-authored-by: inspirations<wangke201@huawei.com> # message auto-generated for no-merge-commit merge: !1480 merge master into master (register)replace socversion with npuArch Created-by: inspirations Commit-by: inspirations Merged-by: cann-robot Description: ## 描述 mc2 3rd目录下的socversion整改 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!1480	3 个月前
matmul_allto_all_tiling_base.cpp	refact arch:根据代际隔离要求重构matmulalltoall Co-authored-by: adamlwang<wangjingquan2@huawei.com> # message auto-generated for no-merge-commit merge: !4952 merge refact into master refact arch:根据代际隔离要求重构matmulalltoall Created-by: adamlwang Commit-by: adamlwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 根据代际隔离要求重构matmulalltoall. ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2327 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR 4952 代码检视报告 ## PR概述 **PR标题**: merge 'refact' into 'master' **变更概要**: 重构 matmul_allto_all 算子 tiling 代码架构，删除工厂类，将工厂逻辑迁移到基类和arch35子类的虚函数中。 **代码侧别**: Tiling侧（Host侧） **检视模式**: PR检视（仅关注变更代码） ## 检视统计 - **检视条款**: 9条重点条款（TOPK问题+红线问题） - **通过条款**: 9条（100%通过） - **发现问题**: 0个高风险问题 - **需关注**: 1个低风险项（已验证） ## 检视结果详情 ### ✅ 通过条款（9条） | 条款ID | 条款标题 | 检视结果 | 说明 | |--------|---------|---------|------| | TOPK-1 | 必须校验函数返回值 | **通过** | 用户反馈已确认第66行返回值问题OK | | TOPK-3 | 避免野指针 | **通过** | 无局部变量指针传递风险 | | TOPK-7 | 外部输入校验 | **通过** | args.rankDim 通过判断函数校验 | | TOPK-8 | GM内存偏移int64 | **通过** | 本次变更无涉及GM地址计算 | | 红线2 | 除零保护 | **通过** | 无除法/求余运算 | | 红线3 | 溢出保护 | **通过** | 无数值溢出风险运算 | | 红线4 | 指针保护 | **通过** | 无指针解引用操作 | | 红线5 | 变量初始化 | **通过** | 继承体系初始化完整 | | 红线6 | 资源配对 | **通过** | 无动态资源申请 | ### 需关注项（低风险，已验证） **关注项1**: formulaicTiling.GetTiling() 返回值未直接使用 **位置**: matmul_allto_all_fit_balance_tiling.h:64 **代码片段**: ```cpp AlltoAllMM formulaicTiling(args, args.rankDim, kernelType, socVersion); formulaicTiling.GetTiling(); // 返回值未使用 return formulaicTiling.tilingM_.cutRes; ``` **风险评估**: 低风险（LOW，置信度20%） - 返回值虽未校验，但下一行直接返回 tilingM_.cutRes - 该设计可能是故意忽略返回值，直接访问成员变量 - 用户反馈已确认之前的问题OK，本关注项仅作记录 ## 变更文件清单 ### 新增文件 - 无 ### 修改文件（8个） 1. mc2/matmul_allto_all/op_host/CMakeLists.txt - 删除tiling编译配置 2. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 3. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.h - 新增虚函数声明 4. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 5. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 6. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 7. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 8. mc2/matmul_allto_all/op_host/op_tiling/arch35/matmul_allto_all_fit_balance_tiling.h - 新增 GetArch35TilingResult() ### 删除文件（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.cpp 2. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.h ### 基类修改（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.cpp - 简化基类实现 2. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.h - 改为虚函数 ## 总体结论 **✅ 检视通过** 本次PR重构符合Ascend C算子开发规范，重点条款100%通过。变更代码： 1. 无数值安全风险（除零、溢出） 2. 无内存安全风险（野指针、未初始化） 3. 无输入验证缺失 4. 无资源管理风险 所有红线问题和TOPK问题均通过检视，代码质量良好。 --- **检视时间**: 2026-05-11 **检视工具**: Ascend C Code Review Agent **规范来源**: ascendc-topk.md + cpp-secure.md See merge request: cann/ops-transformer!4952	22 天前
matmul_allto_all_tiling_base.h	refact arch:根据代际隔离要求重构matmulalltoall Co-authored-by: adamlwang<wangjingquan2@huawei.com> # message auto-generated for no-merge-commit merge: !4952 merge refact into master refact arch:根据代际隔离要求重构matmulalltoall Created-by: adamlwang Commit-by: adamlwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 根据代际隔离要求重构matmulalltoall. ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-transformer/issues/2327 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： # PR 4952 代码检视报告 ## PR概述 **PR标题**: merge 'refact' into 'master' **变更概要**: 重构 matmul_allto_all 算子 tiling 代码架构，删除工厂类，将工厂逻辑迁移到基类和arch35子类的虚函数中。 **代码侧别**: Tiling侧（Host侧） **检视模式**: PR检视（仅关注变更代码） ## 检视统计 - **检视条款**: 9条重点条款（TOPK问题+红线问题） - **通过条款**: 9条（100%通过） - **发现问题**: 0个高风险问题 - **需关注**: 1个低风险项（已验证） ## 检视结果详情 ### ✅ 通过条款（9条） | 条款ID | 条款标题 | 检视结果 | 说明 | |--------|---------|---------|------| | TOPK-1 | 必须校验函数返回值 | **通过** | 用户反馈已确认第66行返回值问题OK | | TOPK-3 | 避免野指针 | **通过** | 无局部变量指针传递风险 | | TOPK-7 | 外部输入校验 | **通过** | args.rankDim 通过判断函数校验 | | TOPK-8 | GM内存偏移int64 | **通过** | 本次变更无涉及GM地址计算 | | 红线2 | 除零保护 | **通过** | 无除法/求余运算 | | 红线3 | 溢出保护 | **通过** | 无数值溢出风险运算 | | 红线4 | 指针保护 | **通过** | 无指针解引用操作 | | 红线5 | 变量初始化 | **通过** | 继承体系初始化完整 | | 红线6 | 资源配对 | **通过** | 无动态资源申请 | ### 需关注项（低风险，已验证） **关注项1**: formulaicTiling.GetTiling() 返回值未直接使用 **位置**: matmul_allto_all_fit_balance_tiling.h:64 **代码片段**: ```cpp AlltoAllMM formulaicTiling(args, args.rankDim, kernelType, socVersion); formulaicTiling.GetTiling(); // 返回值未使用 return formulaicTiling.tilingM_.cutRes; ``` **风险评估**: 低风险（LOW，置信度20%） - 返回值虽未校验，但下一行直接返回 tilingM_.cutRes - 该设计可能是故意忽略返回值，直接访问成员变量 - 用户反馈已确认之前的问题OK，本关注项仅作记录 ## 变更文件清单 ### 新增文件 - 无 ### 修改文件（8个） 1. mc2/matmul_allto_all/op_host/CMakeLists.txt - 删除tiling编译配置 2. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 3. mc2/matmul_allto_all/op_host/op_tiling/arch35/fp_matmul_allto_all_tiling_base.h - 新增虚函数声明 4. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 5. mc2/matmul_allto_all/op_host/op_tiling/arch35/kc_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 6. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.cpp - 新增 GetTilingResult() 7. mc2/matmul_allto_all/op_host/op_tiling/arch35/mx_quant_matmul_allto_all_tiling_base.h - 新增虚函数声明 8. mc2/matmul_allto_all/op_host/op_tiling/arch35/matmul_allto_all_fit_balance_tiling.h - 新增 GetArch35TilingResult() ### 删除文件（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.cpp 2. mc2/matmul_allto_all/op_host/op_tiling/common/allto_all_tiling_factory.h ### 基类修改（2个） 1. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.cpp - 简化基类实现 2. mc2/matmul_allto_all/op_host/op_tiling/matmul_allto_all_tiling_base.h - 改为虚函数 ## 总体结论 **✅ 检视通过** 本次PR重构符合Ascend C算子开发规范，重点条款100%通过。变更代码： 1. 无数值安全风险（除零、溢出） 2. 无内存安全风险（野指针、未初始化） 3. 无输入验证缺失 4. 无资源管理风险 所有红线问题和TOPK问题均通过检视，代码质量良好。 --- **检视时间**: 2026-05-11 **检视工具**: Ascend C Code Review Agent **规范来源**: ascendc-topk.md + cpp-secure.md See merge request: cann/ops-transformer!4952	22 天前