| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
fix: 修复代码规范问题,统一const修饰与字面量规范化 Co-authored-by: ljt<luojiangtao4@huawei.com> # message auto-generated for no-merge-commit merge: !4925 merge master into master fix: 修复代码规范问题,统一const修饰与字面量规范化 Created-by: xiafengkou Commit-by: ljt Merged-by: cann-robot Description: ## 描述 本次改动对 math 目录下相关算子文件进行代码规范修复,**不改变任何原有代码逻辑与算子行为**,未改动任何公开接口与调用方式,确保不影响既有 UT 与 CI 流程。修复内容如下: **1. 只读参数补充 const 修饰(G.CNS.04-CPP,10 处)** - 涉及文件:reduce_log_sum_exp/op_api/aclnn_logsumexp.cpp(CheckShape/CheckPromoteType/CheckDtypeValid)、reduce_var/op_api/aclnn_var.cpp(CheckShape/CheckDtypeValid)、reduce_var/op_api/aclnn_var_mean.cpp(CheckDtypeValid/CheckShape)、addr/op_api/aclnn_addr.cpp(CheckDtypeValid)、cosh/op_api/aclnn_cosh.cpp(CheckInplaceDtypeValid)。 - 方法:为这些只读校验函数中不修改对象本身的 aclTensor* 输出参数添加 const。均为函数内部 static 校验函数的签名调整,调用方无需任何改动。 **2. 字面量提取为命名常量(G.CNS.02-CPP,1 处)** - cross/op_host/arch35/cross_tiling.cpp:cross 维度大小字面量 3 提取为常量 CROSS_DIM_SIZE,消除难以理解的魔法数字。 **3. 其他规范问题(3 处)** - cross/op_host/arch35/cross_tiling.cpp:删除注释掉的死代码 // coreNum = 1;(G.EXP.43-CPP)。 - reduce_mean_with_count/op_api/reduce_mean_with_count.cpp:axisSet.count(d) 改为 axisSet.count(d) > 0,使 if 条件为显式布尔表达式(G.EXP.36-CPP)。 - asinh/op_api/aclnn_asinh.cpp:删除变量声明与 if 之间的冗余空行(G.FMT.17-CPP)。 **4. 成员变量显式初始化(G.CLS.01-CPP,2 处)** - cross/op_host/arch35/cross_tiling.h、polar/op_host/arch35/polar_tiling.h:未初始化的 compileInfo_ 指针成员显式初始化为 nullptr。 ## 关联的Issue 无 ## 测试 - 已通过 IDE 语言诊断检查,无新增编译告警/错误。 - 已通过 git diff 全量复核,确认所有改动均为规范层面的等价改写(const 修饰、字面量常量化、删除死代码/冗余空行、布尔表达式显式化、成员初始化),不影响算子原有计算逻辑与分支。 - 未修改算子 kernel 计算流程与任何公开 API 签名,不影响既有调用方。 ## 文档更新 无文档改动。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:代码规范修复(const 修饰、字面量常量化、删除死代码/冗余空行、布尔表达式显式化、成员变量初始化) See merge request: cann/ops-math!4925 | 2 个月前 | |
modify aclnn dir Co-authored-by: Ding_Jing<dingjing19@huawei.com> # message auto-generated for no-merge-commit merge: !2737 merge asinh_aclnn_0514 into master modify aclnn dir Created-by: Ding_Jing Commit-by: Ding_Jing Merged-by: cann-robot Description: ## 描述 将 asinh 算子的 aclnn 接口文件从 op_host/op_api/ 目录移动到 op_api/ 目录,以符合 ops-math 仓库的目录结构规范。 **变更内容:** 1. 移动文件: - math/asinh/op_host/op_api/aclnn_asinh.cpp → math/asinh/op_api/aclnn_asinh.cpp - math/asinh/op_host/op_api/aclnn_asinh.h → math/asinh/op_api/aclnn_asinh.h - math/asinh/op_host/op_api/asinh.cpp → math/asinh/op_api/asinh.cpp - math/asinh/op_host/op_api/asinh.h → math/asinh/op_api/asinh.h 2. 更新测试文件 include 路径: - math/asinh/tests/ut/op_host/op_api/test_asinh.cpp **原因:** 统一目录结构,将 aclnn 接口与 host 实现分离,职责更加明确,便于后续维护和扩展。 ## 关联的Issue #1557 ## 测试 根据代码变更,测试场景如下: 1. **编译构建测试** - 验证文件移动后项目能正常编译 - 验证 CMake 配置正确识别新路径 2. **单元测试验证** - 执行 math/asinh/tests/ut/op_host/op_api/test_asinh.cpp 单元测试 - 验证 include 路径更新正确,测试通过 3. **功能验证** - 验证 asinh 算子功能正常 - 确认目录结构调整不影响算子行为 ## 文档更新 无需文档更新,此为代码重构,不涉及接口变更。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:♻️ 重构 - 调整目录结构 See merge request: cann/ops-math!2737 | 5 个月前 | |
modify aclnn dir Co-authored-by: Ding_Jing<dingjing19@huawei.com> # message auto-generated for no-merge-commit merge: !2737 merge asinh_aclnn_0514 into master modify aclnn dir Created-by: Ding_Jing Commit-by: Ding_Jing Merged-by: cann-robot Description: ## 描述 将 asinh 算子的 aclnn 接口文件从 op_host/op_api/ 目录移动到 op_api/ 目录,以符合 ops-math 仓库的目录结构规范。 **变更内容:** 1. 移动文件: - math/asinh/op_host/op_api/aclnn_asinh.cpp → math/asinh/op_api/aclnn_asinh.cpp - math/asinh/op_host/op_api/aclnn_asinh.h → math/asinh/op_api/aclnn_asinh.h - math/asinh/op_host/op_api/asinh.cpp → math/asinh/op_api/asinh.cpp - math/asinh/op_host/op_api/asinh.h → math/asinh/op_api/asinh.h 2. 更新测试文件 include 路径: - math/asinh/tests/ut/op_host/op_api/test_asinh.cpp **原因:** 统一目录结构,将 aclnn 接口与 host 实现分离,职责更加明确,便于后续维护和扩展。 ## 关联的Issue #1557 ## 测试 根据代码变更,测试场景如下: 1. **编译构建测试** - 验证文件移动后项目能正常编译 - 验证 CMake 配置正确识别新路径 2. **单元测试验证** - 执行 math/asinh/tests/ut/op_host/op_api/test_asinh.cpp 单元测试 - 验证 include 路径更新正确,测试通过 3. **功能验证** - 验证 asinh 算子功能正常 - 确认目录结构调整不影响算子行为 ## 文档更新 无需文档更新,此为代码重构,不涉及接口变更。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:♻️ 重构 - 调整目录结构 See merge request: cann/ops-math!2737 | 5 个月前 | |
modify aclnn dir Co-authored-by: Ding_Jing<dingjing19@huawei.com> # message auto-generated for no-merge-commit merge: !2737 merge asinh_aclnn_0514 into master modify aclnn dir Created-by: Ding_Jing Commit-by: Ding_Jing Merged-by: cann-robot Description: ## 描述 将 asinh 算子的 aclnn 接口文件从 op_host/op_api/ 目录移动到 op_api/ 目录,以符合 ops-math 仓库的目录结构规范。 **变更内容:** 1. 移动文件: - math/asinh/op_host/op_api/aclnn_asinh.cpp → math/asinh/op_api/aclnn_asinh.cpp - math/asinh/op_host/op_api/aclnn_asinh.h → math/asinh/op_api/aclnn_asinh.h - math/asinh/op_host/op_api/asinh.cpp → math/asinh/op_api/asinh.cpp - math/asinh/op_host/op_api/asinh.h → math/asinh/op_api/asinh.h 2. 更新测试文件 include 路径: - math/asinh/tests/ut/op_host/op_api/test_asinh.cpp **原因:** 统一目录结构,将 aclnn 接口与 host 实现分离,职责更加明确,便于后续维护和扩展。 ## 关联的Issue #1557 ## 测试 根据代码变更,测试场景如下: 1. **编译构建测试** - 验证文件移动后项目能正常编译 - 验证 CMake 配置正确识别新路径 2. **单元测试验证** - 执行 math/asinh/tests/ut/op_host/op_api/test_asinh.cpp 单元测试 - 验证 include 路径更新正确,测试通过 3. **功能验证** - 验证 asinh 算子功能正常 - 确认目录结构调整不影响算子行为 ## 文档更新 无需文档更新,此为代码重构,不涉及接口变更。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:♻️ 重构 - 调整目录结构 See merge request: cann/ops-math!2737 | 5 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 2 个月前 | ||
| 5 个月前 | ||
| 5 个月前 | ||
| 5 个月前 |