0
代码
代码
Issues
Pull Requests
讨论
Wiki
分析
0
cann-robotcann-robotmc2算子op_kernel代码文件代际隔离
5bd4637a创建于 14 天前历史提交
文件最后提交记录最后更新时间
kc_quant_matmul_allto_all_arch35.h
move template Co-authored-by: qq_43844249<fanglin17@huawei.com> # message auto-generated for no-merge-commit merge: !2662 merge move_mc2template_to_common into master move template Created-by: qq_43844249 Commit-by: qq_43844249 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 将matmulalltoall算子下模板库迁移至common目录，下沉不能算子间复用的alltoall流水线 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!26621 个月前
matmul_allto_all_apt.cpp
mc2算子op_kernel代码文件代际隔离 Co-authored-by: hblnb<huangbolin3@huawei.com> # message auto-generated for no-merge-commit merge: !1830 merge agmv2_csv into master mc2算子op_kernel代码文件代际隔离 Created-by: hblnb Commit-by: hblnb Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!183014 天前
matmul_allto_all_arch35.h
move template Co-authored-by: qq_43844249<fanglin17@huawei.com> # message auto-generated for no-merge-commit merge: !2662 merge move_mc2template_to_common into master move template Created-by: qq_43844249 Commit-by: qq_43844249 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 将matmulalltoall算子下模板库迁移至common目录，下沉不能算子间复用的alltoall流水线 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!26621 个月前
matmul_allto_all_pipeline.h
move template Co-authored-by: qq_43844249<fanglin17@huawei.com> # message auto-generated for no-merge-commit merge: !2662 merge move_mc2template_to_common into master move template Created-by: qq_43844249 Commit-by: qq_43844249 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 将matmulalltoall算子下模板库迁移至common目录，下沉不能算子间复用的alltoall流水线 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!26621 个月前
matmul_allto_all_tiling_data.h
调整common目录结构 Co-authored-by: hello_simida<wangyi206@huawei.com> # message auto-generated for no-merge-commit merge: !2465 merge master_v3 into master 调整common目录结构 Created-by: hello_simida Commit-by: hello_simida Merged-by: cann-robot Description: ## 描述 调整common目录调整 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-transformer!24652 个月前
matmul_allto_all_tiling_key.h
[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!518220 天前
mx_quant_matmul_allto_all_arch35.h
move template Co-authored-by: qq_43844249<fanglin17@huawei.com> # message auto-generated for no-merge-commit merge: !2662 merge move_mc2template_to_common into master move template Created-by: qq_43844249 Commit-by: qq_43844249 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 将matmulalltoall算子下模板库迁移至common目录，下沉不能算子间复用的alltoall流水线 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [x ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/ops-transformer!26621 个月前