已合并
feat: rename op_api_def.h to op_api_def_nn.h #8360
FishPotatoChen创建于 20 天前
feat: rename op_api_def.h to op_api_def_nn.h #8360
已合并
FishPotatoChen创建于 20 天前
FishPotatoChen
20 天前

描述

本次改动将 op_api/op_api_def.h 重命名为 op_api/op_api_def_nn.h,并同步更新了所有引用该头文件的源文件与测试文件中的 #include 路径,共涉及 161 个文件。

改动原因:将 op_api 定义头文件统一命名为 op_api_def_nn.h,以更好地体现其所属模块(NN 算子)的语义,便于后续与其他模块的头文件区分管理。

此外,在提交过程中发现 4 个文件(aclnn_foreach_addcmul_scalar_v2.cppaclnn_max_unpool3d.cpptest_aclnn_addmv.cpptest_aclnn_fast_batchnorm_backward.cpp)的许可证头使用了非标准的旧格式,导致 OAT 合规检查失败。本次一并将其更新为标准的 CANN 开源许可证头格式,确保通过合规校验。

关联的Issue

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

测试

  • 通过 pre-commit 钩子校验:trailing whitespace、end-of-file-fixer、clang-format 均通过
  • 通过 OAT 合规检查(161 个文件全部通过)
  • 本次为头文件重命名改动,不涉及逻辑变更,编译依赖关系保持不变

文档更新

类型标签

AI/Agent生成声明

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 FishPotatoChen 的贡献)
FishPotatoChenFishPotatoChen
20 天前 创建了 pull request,commit 44e2d697
atomgit-bot
atomgit-bot
20 天前 评论:

变更摘要

本次改动将 op_api/op_api_def.h 重命名为 op_api/op_api_def_nn.h,以更好地体现其所属 NN 算子模块的语义,便于与后续其他模块的定义头文件区分管理。同步更新了所有引用该头文件的约 160 个源文件与测试文件中的 #include 路径,同时修复了 4 个文件中非标准的旧格式许可证头,以及多处文件末尾缺失换行符的格式问题。本次改动不涉及任何逻辑变更,编译依赖关系保持不变。

主要改动

  • 头文件重命名及守卫宏更新:将 common/inc/op_api/op_api_def.h 重命名为 common/inc/op_api/op_api_def_nn.h,头文件包含守卫从 OP_API_DEF_H_ 同步更新为 OP_API_DEF_NN_H_
  • 全量 #include 路径同步:将所有层级的算子实现文件(activation、conv、norm、loss、matmul、pooling、index、foreach、rnn、quant 等模块)以及 common/inc/op_api/level2_base.hlevel2_base_caculation.hlevel2_base_loss.h 等公共头文件中的 #include "op_api/op_api_def.h" 统一改为 #include "op_api/op_api_def_nn.h"
  • 许可证头合规修复:修复 aclnn_foreach_addcmul_scalar_v2.cppaclnn_max_unpool3d.cpptest_aclnn_addmv.cpptest_aclnn_fast_batchnorm_backward.cpp 四个文件中不符合 OAT 合规检查的旧格式许可证头,统一替换为标准的 CANN 开源许可证头格式
  • 文件末尾格式修复:修正大量 .cpp 文件中文件末尾 #endif 后缺失的换行符(即 \ No newline at end of file 问题),确保通过 end-of-file-fixer 校验
  • 少量代码格式微调:在 aclnn_softshrink_backward.cppaclnn_convolution_backward.cpp 中修正了代码换行风格及注释前的空格格式
likedislike
atomgit-bot
atomgit-bot
20 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
20 天前 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
20 天前 添加了label:cann-cla/yes
此处折叠了163条消息 查看更多
wangcimeng成员
5 天前 评论:

/lgtm
/approve

likedislike
liuyang_806成员
5 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
5 天前 添加了label:approved
CANN-robotCANN-robot成员
5 天前 关闭了关联的issue
CANN-robotCANN-robot成员
5 天前 合入了pull request