已合并
refactor: 为公共op_api头文件增加NN后缀 #10791
refactor: 为公共op_api头文件增加NN后缀 #10791
已合并
clinglai0517创建于 7 天前
clinglai0517成员
7 天前

描述

ops-nn 的 common/inc/op_api 目录中仍有 level2_base.h 和 level2_base_caculation.h 两个通用名称。它们与其他算子仓的同名文件一起进入软件包时存在覆盖风险。

本 MR 是 ops-nn 分阶段迁移的第一阶段:

  • 新增完整实现头 level2_base_nn.h 和 level2_base_caculation_nn.h;
  • 保留原文件为轻量转发头,兼容尚未迁移的存量 ACLNN 源文件和开发中的 MR;
  • 新计算头只依赖新基础头,避免新命名链路回退到旧文件名;
  • op_api_def_nn.h 已具有 NN 标识,本次不修改;
  • classify_rule.yaml 中没有这两个头文件的现有条目,也没有证据表明该文件负责软件包头文件筛选,因此本次不修改;
  • 先迁移 Mish、UniqueConsecutive、Silu、LogSigmoid 四个简单 ACLNN 接口作为样例。

本阶段不删除兼容头,也不声明已完成最终软件包去重。当前仍有 32 处 level2_base.h 和 5 处 level2_base_caculation.h 的旧引用,后续可按模块分批迁移;旧引用归零并验证实际制品清单后,再移除兼容头。

关联的Issue

关联并在合入后关闭 #6008:

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

测试

已完成:

  • git diff --check;
  • 新旧完整实现内容等价性检查(仅头文件保护宏和内部 include 路径变化);
  • 两个旧文件均为 16 行轻量转发头;
  • 新命名头文件之间仅通过新文件名依赖;
  • 四个样例 ACLNN 文件均为单行 include 替换;
  • 使用 clang++ -M -MG 完成头文件依赖解析冒烟检查;
  • 统计剩余旧引用:level2_base.h 32 处、level2_base_caculation.h 5 处;
  • 8 个变更文件通过 OAT 合规检查;
  • origin 同名分支与本地提交一致。

文档更新

无。

类型标签

AI/Agent生成声明

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 clinglai0517 的贡献)
Cclinglai0517成员
7 天前 创建了 pull request,commit 0ffcfe0a
Cclinglai0517成员
7 天前 关联了issue:[Requirement|需求建议]: ops-nn公共op_api头文件增加NN标识
CANN-robotCANN-robot成员
7 天前 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
7 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7 天前 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here.
You can get sig-info at here.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
*/*/op_api/*.h ✅ 汤磊, 陈琦, 汤平川 (3/2) ✅ 汤磊 (1/1)
activation ✅ 汤平川, 陈琦 (2/2) ✅ 汤平川 (1/1)
index ✅ 汤平川, 陈琦 (2/2) ✅ 汤平川 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

clinglai0517, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
此处折叠了49条消息 查看更多
yuning_chen
yuning_chen成员
6 天前 评论:

您好,pr合入已准备就绪,请尽快联系committer进行检视,谢谢!

likedislike
TangPC
TangPC成员
6 天前 评论:

/lgtm
/approve

likedislike
CANN-robot
CANN-robot成员
6 天前 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
/approve: 陈琦

likedislike
CANN-robotCANN-robot成员
6 天前 添加了label:lgtmapproved
CANN-robotCANN-robot成员
6 天前 合入了pull request