已合并
fix(om2): 重命名 runtime/om2 下与 base/ 重复的头文件和源文件 #4470
fix(om2): 重命名 runtime/om2 下与 base/ 重复的头文件和源文件 #4470
已合并
wuzheng创建于 13 天前
wuzheng成员
13 天前

Pull Request

描述

提交 f35963b72 引入了变量格式转换和类型转换功能,在 runtime/om2/common/runtime/om2/formats/ 目录下新增了头文件和源文件。这些文件是从 base/common/base/formats/ 拷贝的,导致仓库中出现相同名称的头文件(24对重复文件),header guard 也完全相同。

本次修改:

  • runtime/om2/ 下 24 对重复的 .h/.cc 文件重命名,添加 om2_ 前缀
  • 更新所有 #include 引用路径指向新文件名
  • 更新 header guard 宏名,添加 OM2_ 前缀消除与 base/ 版本的冲突
  • 已验证 ge_executorge_compiler 构建通过

变更类型

关联的Issue

如何测试

  1. 构建 ge_executor 和 ge_compiler,确认编译通过
  2. 检查仓库中不再存在同名头文件(base/ 和 runtime/om2/ 下)
  3. 运行 om2 相关 UT 用例确认功能不受影响

核对清单

其他信息

涉及 48 个文件变更(24 对 .h/.cc 文件重命名 + include 路径和 header guard 更新)。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 wuzheng 的贡献)
Wwuzheng成员
13 天前 创建了 pull request,commit 6adc959d
atomgit-bot
atomgit-bot
13 天前 评论:

变更摘要

本 PR 修复 runtime/om2/ 下与 base/ 目录重名文件导致的编译冲突问题。此前提交引入的变量格式转换/类型转换功能将 base/common/base/formats/ 下的文件拷贝至 runtime/om2/,造成 24 对同名 .h/.cc 文件且 header guard 完全相同的重复。本次为这些文件添加 om2_ 前缀进行重命名,并同步更新所有 #include 引用路径与 header guard 宏名(添加 OM2_ 前缀),已验证 ge_executorge_compiler 构建通过。

主要改动

  • 文件重命名消除同名冲突:将 runtime/om2/common/(如 datatype_transfer.hom2_datatype_transfer.hfp16_t.hom2_fp16_t.hhif8_t.hom2_hif8_t.h)及 runtime/om2/formats/(如 formats.hom2_formats.hregister_format_transfer.hom2_register_format_transfer.hformat_transfer_*.hom2_format_transfer_*.h)下共 24 对头文件/源文件重命名,使仓库中不再存在与 base/ 同名文件。
  • 更新 #include 引用路径:所有源文件(如 om2_datatype_transfer.cc、各 om2_format_transfer_*.ccom2_formats.ccom2_register_format_transfer.cc)及引用方 runtime/om2/om2_rt_var_manager.cc 中的 #include 均改为指向新文件名,例如 "formats/formats.h""formats/om2_formats.h""formats/utils/formats_trans_utils.h""formats/utils/om2_formats_trans_utils.h"
  • 更新 header guard 宏名:所有重命名头文件的 header guard 由 GE_COMMON_*/INC_REGISTER_* 统一改为 GE_OM2_COMMON_*/INC_OM2_*(如 GE_COMMON_FORMATS_FORMATS_H_GE_OM2_COMMON_FORMATS_FORMATS_H_INC_REGISTER_REGISTER_FORMAT_TRANSFER_H_INC_OM2_REGISTER_REGISTER_FORMAT_TRANSFER_H_),消除与 base/ 版本宏的冲突。
  • 交叉引用同步重定向:重命名后的头文件内部相互引用的 include 也同步更新,如 om2_datatype_transfer.hom2_formats_trans_utils.h 改引 formats/om2_register_format_transfer.hom2_hif8_t.h 改引 common/fp16_t/om2_fp16_t.hom2_format_transfer_fz_c04.cc 改引 formats/om2_formats.h 等,保证重命名后依赖链完整一致。
likedislike
atomgit-bot
atomgit-bot
13 天前 评论:

代码审查

✅ 未发现问题

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

CLA Signature Pass

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

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

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.
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
repo-cann/ge 张德鹏, kobemini, stevenaw0 (3/2) 张德鹏 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
13 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
13 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
wuzheng成员
13 天前 评论:

compile

likedislike
Wwuzheng成员
13 天前 update merge request[project id: 8824148, iid: 4470, commit_id: dac16c23ef59b718c8332700a348bd7510a88c8c] virtual merging success
CANN-robotCANN-robot成员
13 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
13 天前 评论:

流水线任务触发成功
任务链接 [3ee3d155d5a74a3aae766b2787688c76][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_ARM_compiler ✅ SUCCESS >>>>> >>>>>
Compile_ARM_executor ✅ SUCCESS >>>>> >>>>>
Compile_ARM_dflow ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-08-20 10:54:11]    CI执行结束

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

流水线任务触发成功
任务链接 [f6a63be8c94941a8af07ac7a51e7c4d5][流水线指导]

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_X86_compiler ✅ SUCCESS >>>>> >>>>>
Compile_X86_executor ✅ SUCCESS >>>>> >>>>>
Compile_X86_dflow ✅ SUCCESS >>>>> >>>>>
UT_Test_dflow ✅ SUCCESS
UT_Test_ge_common ✅ SUCCESS
UT_Test_parser ✅ SUCCESS
UT_Test_python ✅ SUCCESS >>>>>
UT_Test_rt ✅ SUCCESS
UT_Test_acl ✅ SUCCESS
UT_Test_autofuse ✅ SUCCESS
UT_Test_executor ✅ SUCCESS
UT_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_dflow ✅ SUCCESS
ST_Test_ge_common ✅ SUCCESS
ST_Test_parser ✅ SUCCESS
ST_Test_python ✅ SUCCESS >>>>>
ST_Test_rt ✅ SUCCESS
ST_Test_autofuse ✅ SUCCESS
ST_Test_executor ✅ SUCCESS
ST_Test_hetero ✅ SUCCESS
ST_Test_autofuse_ascendc_api ✅ SUCCESS
ST_Test_autofuse_e2e ✅ SUCCESS
UT_Test_fe ✅ SUCCESS
UT_Test_tefusion ✅ SUCCESS
ST_Test_fe ✅ SUCCESS
ST_Test_tefusion ✅ SUCCESS
ST_Test_ge_common_atc ✅ SUCCESS
UT_Test_Report_graphengine ✅ SUCCESS >>>>>
UT_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
UT_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
ST_Test_Report_graphengine ✅ SUCCESS >>>>>
ST_Test_Report_ge_executor_c ✅ SUCCESS >>>>>
ST_Test_Report_ge_autofuse ✅ SUCCESS >>>>>
UT_Test_Report_fe ✅ SUCCESS >>>>>
ST_Test_Report_fe ✅ SUCCESS >>>>>

[2026-08-20 11:01:04]    CI执行结束

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

流水线任务触发成功
任务链接 [3b7f03a5868c49449ded13260db2c158][流水线指导]

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-08-20 10:51:29]    CI执行结束

likedislike
CANN-robotCANN-robot成员
13 天前 添加了label:api-check-pass
CANN-robotCANN-robot成员
13 天前 删除了label:api-check-pass
CANN-robotCANN-robot成员
13 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
13 天前 添加了label:ci-pipeline-passed
GengChao
GengChao成员
13 天前 评论:

/lgtm

likedislike
黄桂军
黄桂军成员
13 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
13 天前 添加了label:lgtm
张德鹏成员
13 天前 评论:

/approve

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