已合并
fix:重构runtime2_util.h #8841
FishPotatoChen创建于 15 天前
fix:重构runtime2_util.h #8841
已合并
FishPotatoChen创建于 15 天前
FishPotatoChen
15 天前

描述

common/inc/op_api/runtime2_util.h 重命名为 common/inc/op_api/runtime2_util_nn.h,以避免与 CANN 仓内其他模块同名头文件产生命名冲突。

主要改动内容:

  1. 重命名头文件 runtime2_util.hruntime2_util_nn.h,并同步更新文件内的 doxygen 注释、头文件保护宏(CANN_OPS_BUILT_IN_OP_TILING_RUNTIME2_UTIL_NN_H_)以及文件末尾换行。
  2. 批量更新所有引用该头文件的 26 个源文件(.h/.cpp)中的 #include "op_api/runtime2_util.h"#include "op_api/runtime2_util_nn.h",涉及 activation、index、loss、norm 等 nn 算子目录。
  3. 同步更新 common/inc/op_host/cache_runinfo.h 等公共头文件的引用,并更新版权年份至 2025-2026。

该改动为纯重命名重构,不涉及任何功能逻辑变更。

关联的Issue

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

测试

  • 由于改动仅涉及头文件重命名及引用路径更新,无功能逻辑变更,主要通过全量编译验证所有受影响算子可正确找到新头文件并编译通过。
  • 编译范围覆盖 activation、index、loss、norm 等目录下受影响的 arch35 tiling 与 infershape 模块。

文档更新

类型标签

AI/Agent生成声明

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

变更摘要

本 PR 为纯重命名重构:将 common/inc/op_api/runtime2_util.h 重命名为 common/inc/op_api/runtime2_util_nn.h,以避免与 CANN 仓内其他模块的同名头文件产生命名冲突,并同步更新全部引用方,不涉及任何功能逻辑变更。

主要改动

  • 头文件重命名: 将 common/inc/op_api/runtime2_util.h 重命名为 common/inc/op_api/runtime2_util_nn.h,同步更新文件内 doxygen 的 \file 注释、头文件保护宏(CANN_OPS_BUILT_IN_OP_TILING_RUNTIME2_UTIL_H_CANN_OPS_BUILT_IN_OP_TILING_RUNTIME2_UTIL_NN_H_)以及文件末尾换行。
  • 批量更新引用: 将 26 个源文件(.h/.cpp)中的 #include "op_api/runtime2_util.h" 统一改为 #include "op_api/runtime2_util_nn.h",覆盖 activation、index、loss、norm 等 nn 算子目录下的 arch35 tiling 与 infershape 模块。
  • 公共头文件同步更新: 更新 common/inc/op_host/cache_runinfo.h 对重命名头文件的引用,并将相关文件的版权年份由 2025 更新为 2025-2026。
  • 附带格式微调: 补齐多个文件末尾缺失的换行符,并统一 group_norm_v2_tiling_arch35.cppOP_CHECK_IF 调用(如 return );return);)及 cache_runinfo.h 中构造函数 GenericHashItem(){} 的写法,均无逻辑影响。
likedislike
atomgit-bot
atomgit-bot
15 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
15 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
15 天前 评论:

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

⚠️ This PR does not yet meet the following requirements:lgtm (requires ≥ 2 person(s) per module)、approve (requires ≥ 1 person(s) per module)

Module Approval Details

module lgtm status approve status
*/*/op_api/*.h ❌ (0/2)(You can also ask: 陈琦, 汤平川, 杨阳, 曾娟, 胡碧霞) ❌ (0/1)(You can also ask: 汤磊, 王永光)
activation ❌ (0/2)(You can also ask: 曾娟, 黄迪, 刘波, 王瑞, 唐超) ❌ (0/1)(You can also ask: 钱泽洪, 任如海, 刘杰, 王星, 查建青)
common ❌ (0/2)(You can also ask: 唐玮玮, 陈琦, 杨阳, 汤平川, 刘波) ❌ (0/1)(You can also ask: 杨阳, 商晓波, 林鹏翔, 范其瑞, 唐玮玮)
experimental ❌ (0/2)(You can also ask: 陈华锋, 查建青, 苏跃明, 沈承宗, 刘闯) ❌ (0/1)(You can also ask: 王慈孟, 苏跃明, 余鑫, 郑李磊, 陈昊文)
index ❌ (0/2)(You can also ask: 唐玮玮, 王瑞, 陈风, 唐超, 唐燕峰) ❌ (0/1)(You can also ask: 黄迪, 王星, 汤平川, 唐燕峰, 苏跃明)
loss ❌ (0/2)(You can also ask: 胡碧霞, 杨阳, 曾娟, 刘波, 汤平川) ❌ (0/1)(You can also ask: 汤平川, 钱泽洪, 於欣洁, 陈风, 王星)
norm ❌ (0/2)(You can also ask: 王瑞, 钱泽洪, 陈昊文, 查建青, 刘波) ❌ (0/1)(You can also ask: 查建青, 苏跃明, 王瑞, 黄迪, 王星)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了114条消息 查看更多
CANN-robotCANN-robot成员
9 天前 添加了label:api-check-failed
CANN-robotCANN-robot成员
9 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
9 天前 添加了label:ci-pipeline-passed
CANN-robotCANN-robot成员
9 天前 关闭了关联的issue
CANN-robotCANN-robot成员
9 天前 合入了pull request