master

cann-robotrefactor: 统一 math 目录 13 个算子的代码风格与格式规范

20ac2f7aAdd on 23 days agoHistory

Files	Last commit	Last update
aclnn_div.cpp	refactor: 为公共op_api头文件增加math后缀 Co-authored-by: clinglai0517<laichangling@huawei.com> # message auto-generated for no-merge-commit merge: !5839 merge refactor/math-prefix-common-headers into master refactor: 为公共op_api头文件增加math后缀 Created-by: clinglai0517 Commit-by: clinglai0517 Merged-by: cann-robot Description: ## 描述 ops-math/common/inc/op_api 中的 level2_base.h、level2_base_caculation.h、op_api_def.h 名称缺少仓库标识。在软件包合并交付或公共 include 目录集成时，存在被其他仓库同名文件覆盖，或因 include 路径顺序产生误引用的风险。 本 MR 参考同目录已有的 op_resource_math.h 命名，为三个公共头文件增加 _math 后缀，并通过兼容转发头分阶段迁移现有引用。 ## 修改内容 - 新增 level2_base_math.h、level2_base_caculation_math.h、op_api_def_math.h，作为带仓库标识的正式实现； - 原 level2_base.h、level2_base_caculation.h、op_api_def.h 保留为轻量兼容转发头，避免未迁移代码立即失效； - 新头文件之间的内部依赖统一使用 _math 后缀路径； - 更新 classify_rule.yaml，让 level2_base_math.h 继承原组件及 UT 覆盖率分类； - 将 89 处旧头文件引用迁移到新名称： - level2_base.h：20 处； - level2_base_caculation.h：9 处； - op_api_def.h：60 处； - 当前 MR 共修改 81 个文件，新增 434 行、删除 384 行。 为减少无关格式变化和后续合并冲突，本次未迁移会触发额外 clang-format、尾部空白或 EOF 修复，或组合触发既有 UT 隔离问题的文件。这些文件将拆分到后续 MR 逐步处理。 ## 剩余迁移 当前仓内仍有 67 处旧引用，分布在 65 个唯一文件中，继续通过兼容转发头工作： - level2_base.h：24 个文件； - level2_base_caculation.h：2 个文件； - op_api_def.h：41 个文件。 待旧引用分批迁移完成后，再结合实际打包和安装规则移除兼容转发头。本 MR 不移除旧头文件，也不声明已完成最终软件包去重。 ## 关联的 Issue 关联并在合入后关闭 #3243： https://gitcode.com/cann/ops-math/issues/3243 ## 测试 已完成： - 对本轮新增的 69 个纯引用替换文件逐行检查，80 组增删均仅修改目标 include，无其他代码或格式变化； - 额外纳入 aclnn_log10.cpp 和 aclnn_logdet.cpp，每个文件最终均为 3 行新增、3 行删除，低于 10 行修改阈值；math/sin/op_api/aclnn_sin.cpp 因聚合 UT 存在平台状态泄漏暂缓迁移； - 在 CANN 9.0.0 容器执行完整 pre-commit，尾部空白、EOF、clang-format、codespell 和 OAT 等检查全部通过； - 在 CANN 9.0.0 容器执行 bash build.sh --opapi -j32，全量构建通过； - 成功生成 build/libopapi_math.so，SHA-256：abaa8f8408979d931a8dad09f8e06875722fc42bbdddc72aaf9f14944bc163ea； - classify_rule.yaml 可正常解析，三个旧文件继续作为兼容转发头使用。 ## 文档更新 不涉及用户文档变更。 ## 类型标签 - [ ] Bug 修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他：代码重构 See merge request: cann/ops-math!5839	1 month ago
aclnn_div.h	refactor: 统一 math 目录 13 个算子的代码风格与格式规范 Co-authored-by: cai-chengchao<caichengchao1@h-partners.com> # message auto-generated for no-merge-commit merge: !6542 merge ccc_1020 into master refactor: 统一 math 目录 13 个算子的代码风格与格式规范 Created-by: cai-chengchao Commit-by: cai-chengchao Merged-by: cann-robot Description: ## 描述 本 PR 对 math 目录下 13 个算子进行代码风格规范化（代码格式化），不涉及功能性改动。涉及的算子包括：axpy、axpy_v2、bias_add、div、div_no_nan、gcd、ger、muls、select、select_v2、signbit、squared_difference、tanh_grad。 ### 改动原因 统一算子代码的编码风格与格式规范，提升代码可读性和可维护性。 ### 改动方法 对 C++ 源码、Python 脚本及 CMake 构建文件进行格式化，具体包括： - 统一 License 头注释格式（分隔线长度与换行） - 统一命名空间括号风格（namespace xxx { 同行） - 统一 // namespace xxx 尾部注释与 #endif // xxx 注释格式（单空格） - 移除行尾空格，补全文件末尾换行 - 统一缩进（2 空格 → 4 空格）与函数声明/定义换行 - 统一 Python 脚本 docstring 引号与长语句换行 - 统一 CMakeLists 文件 License 头与文件末尾换行 ## 关联的Issue - #3814 ## 测试 本次改动为纯代码格式化，无功能性变更，逻辑行为保持不变。 ## 文档更新 无 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：代码风格规范化（格式化） See merge request: cann/ops-math!6542	23 days ago
div.cpp	feat: 算子910B~910E区间判断追加IsRegBase()并统一DAV_3510硬编码判断以兼容后续Regbase芯片 Co-authored-by: hahaha22<wangrunze20@h-partners.com> # message auto-generated for no-merge-commit merge: !5467 merge q_aclnn_version into master feat: 算子910B~910E区间判断追加IsRegBase()并统一DAV_3510硬编码判断以兼容后续Regbase芯片 Created-by: hahaha22 Commit-by: hahaha22 Merged-by: cann-robot Description: ## 描述 以 RegBase 架构（NpuArch = DAV_3510，含 Ascend950 及后续 Regbase 芯片）兼容为目标，统一两类芯片判断写法： 1. **区间判断追加**：对 13 个算子 op_api 中 GetSocVersion() >= ASCEND910B && GetSocVersion() <= ASCEND910E 区间判断追加 || IsRegBase()，使 RegBase 平台在 SocVersion 区间之外时也复用 910B 支持分支（放开 BF16 dtype 校验） 2. **硬编码统一**：将 11 个算子文件中 case NpuArch::DAV_3510: 硬编码判断改为 IsRegBase() 谓词，后续 Regbase 芯片（IsRegBase 集合扩展新 arch 值时）无需逐文件修改即可自动命中 ## 修改内容一：区间判断追加（13 个文件，15 处） | 算子 | 文件 | 修改位置 | |------|------|----------| | less | math/less/op_api/aclnn_lt_scalar.cpp | GetDtypeSupportList、GetOutDtypeSupportList（2 处） | | less | math/less/op_api/aclnn_lt_tensor.cpp | GetDtypeSupportList、GetOutDtypeSupportList（2 处） | | less_equal | math/less_equal/op_api/aclnn_le_scalar.cpp | GetOutputDtypeSupportList（1 处） | | logical_or | math/logical_or/op_api/aclnn_logical_or.cpp | CheckSocVersionIsSupportBf16（1 处） | | maximum | math/maximum/op_api/aclnn_maximum.cpp | GetDtypeSupportList（1 处） | | minimum | math/minimum/op_api/aclnn_minimum.cpp | GetDtypeSupportList（1 处） | | mul | math/mul/op_api/aclnn_mul.cpp | GetDtypeSupportList（1 处） | | not_equal | math/not_equal/op_api/aclnn_logical_xor.cpp | CheckSocVersionIsSupportBf16（1 处） | | pow | math/pow/op_api/aclnn_pow.cpp | CheckSocVersionIsSupportBf16（1 处） | | pow | math/pow/op_api/aclnn_pow_tensor_tensor.cpp | CheckSocVersionIsSupportBf16（1 处） | | signbit | math/signbit/op_api/aclnn_signbit.cpp | GetDtypeSupportList（1 处） | | sub | math/sub/op_api/sub.cpp | IsAiCoreSupport（1 处） | | equal | math/tensor_equal/op_api/aclnn_equal.cpp | CheckSocVersionGe910B（1 处） | 其中 sub.cpp、aclnn_pow_tensor_tensor.cpp、aclnn_logical_xor.cpp、aclnn_equal.cpp 补充 #include "op_api/aclnn_check.h"。 pow.cpp 的 IsAiCoreSupport、aclnn_ne_scalar.cpp、aclnn_ne_tensor.cpp：区间判断前已有 if (IsRegBase()) 提前返回分支（追加会造成死条件），经评审不修改。 ## 修改内容二：硬编码判断统一（11 个文件，13 处） ### A类：switch 双 case（DAV_3510 与 DAV_2201 同分支，8处） switch 前置 if (IsRegBase(curArch)) 返回原 DAV_3510 分支列表，删除 case NpuArch::DAV_3510: fall-through 行： - Add：math/add/op_api/add.cpp、aclnn_add.cpp - Sub：math/sub/op_api/aclnn_sub.cpp、aclnn_rsub.cpp - div：math/div/op_api/div.cpp（补 include） - floordiv：math/floor_div/op_api/floordiv.cpp（补 include） - tanhgrad：math/tanh_grad/op_api/aclnn_tanh_backward.cpp - floormod：math/floor_mod/op_api/aclnn_remainder.cpp ### B类：switch 独立 case（DAV_3510 单独返回 REGBASE 专用列表，3处） switch 前置 if (IsRegBase(curArch)) 返回 REGBASE 列表，删除独立 case 块： - mul：math/mul/op_api/mul.cpp（REGBASE_AICORE_DTYPE_SUPPORT_LIST） - equal：math/equal/op_api/aclnn_eq_tensor.cpp、aclnn_eq_scalar.cpp（REGBASE_DTYPE_SUPPORT_LIST） ## 关联的Issue - #3100 （math仓910B~910E区间判断的RegBase兼容遗留清理：isclose恒真条件bug、8个同类判断文件未覆盖IsRegBase） ## 测试 - pre-commit 检查（clang-format/codespell/OAT 等）全部通过（含对修改文件的全量格式化） - 全部修改文件 g++ -fsyntax-only 语法验证通过 - 已触发 compile 编译 - 建议测试项： 1. RegBase（DAV_3510）平台验证上述算子 BF16 输入/输出 dtype 校验与 910B 平台一致 2. Ascend950 平台回归 A/B 类算子 dtype 选择行为不变 3. 回归 910B~910E、910/1980、310/310P 平台行为不受影响 ## 文档更新 无 ## 类型标签 - [ ] Bug修复 - [x] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： ### 关联issue - [cann/ops-math#5512（报错 dtype 支持列表双层中括号修复，同改 aclnn_remainder.cpp/eq 系列）](https://gitcode.com/cann/ops-math/pull/5512) - [cann/ops-math#5519（RemainderScalarTensor Regbase 计算类型校验，同改 aclnn_remainder.cpp）](https://gitcode.com/cann/ops-math/pull/5519) - [cann/ops-nn#10101（ops-nn 侧同类 Regbase 判断统一）](https://gitcode.com/cann/ops-nn/pull/10101) See merge request: cann/ops-math!5467	2 months ago
div.h	支持Div算子在下一代实现 Co-authored-by: yuhwang<wangyuhang70@huawei.com> # message auto-generated for no-merge-commit merge: !654 merge master into master 支持Div算子在下一代实现 Created-by: yuhwang Commit-by: yuhwang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!654	10 months ago