已合并
refactor: 删除 ge_rts_decl.h 中不再使用的日落接口 #5127
refactor: 删除 ge_rts_decl.h 中不再使用的日落接口 #5127
已合并
tangqunzhang创建于 13 天前
tangqunzhang成员
13 天前

Pull Request

描述

删除 inc/common/ge_rts_decl.h 中生产代码已不再使用的日落接口声明,并同步清理依赖这些符号的 UT/ST stub。

删除的接口/定义:

  • rtVectorCoreKernelLaunchWithHandle、rtVectorCoreKernelLaunch:仅测试 stub 定义,无任何生产调用
  • rtPlaceHolderInfo_t、rtCpuKernelArgs_t:生产已改用 aclrtPlaceHolderInfo 等新接口,全仓无引用

同步整改:

  • tests/depends/runtime/src/runtime_stub.h/.cc:删除对应虚方法及符号定义
  • tests/framework/ge_runtime_stub/.../ge_fake_launch_args.h/.cc:删除使用已删类型的未调用构造函数

变更类型

关联的Issue

无

如何测试

描述测试此变更的步骤和前提条件:

  1. cmake 配置后编译 graph_base、runtime_stub、ge_runtime_stub、gert 均通过
  2. 运行 sink_node_bin 相关 UT/ST 及 runtime stub 相关用例

核对清单

其他信息

本次改动为纯删除,不改变任何运行时行为。相关符号已确认无任何调用方(llt_hccl_stub.h 中同名类型为其自带独立定义,不受影响)。

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

变更摘要

本次变更是一次纯删除型重构,目标是清理 inc/common/ge_rts_decl.h 中生产代码已不再使用的日落接口声明与类型定义,并同步移除依赖这些符号的测试桩实现,不改变任何运行时行为。删除内容包括 rtVectorCoreKernelLaunchWithHandle、rtVectorCoreKernelLaunch 两个 RTS_API 接口声明,以及 rtPlaceHolderInfo_t、rtCpuKernelArgs_t 两个类型定义(生产侧已改用 aclrtPlaceHolderInfo 等新接口)。同时清理了 tests/depends/runtime 与 tests/framework/ge_runtime_stub 中对应的桩方法与构造函数。

主要改动

  • 删除日落接口声明: 从 ge_rts_decl.h 移除 rtVectorCoreKernelLaunchWithHandle、rtVectorCoreKernelLaunch 两个 RTS_API 函数声明,以及 rtPlaceHolderInfo_t、rtCpuKernelArgs_t 两个结构体类型定义。
  • 清理 runtime stub 实现与虚方法: 在 runtime_stub.cc 中删除上述两个接口的桩函数定义,在 runtime_stub.h 中同步删除 RuntimeStub 类内对应的两个 virtual 方法。
  • 清理 ge_runtime_stub 未调用构造函数: 在 ge_fake_launch_args.h/.cc 中删除使用 rtCpuKernelArgs_t、rtPlaceHolderInfo_t 的两个 GeFakeLaunchArgs 构造函数声明与实现。
likedislike
不准确?
atomgit-bot
atomgit-bot
13 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
13 天前 添加了label:cann-cla/yes
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.
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
repo-cann/ge ✅ yangyongqiang0606, tang-haojie, yuht9 (3/2) ✅ yangyongqiang0606 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robot
CANN-robot成员
13 天前 评论:

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
13 天前 将yangyongqiang0606,kobemini,ClarkXie,xchu42,hugo111,lining23666,tang-haojie,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
13 天前 将yangyongqiang0606,kobemini,ClarkXie,xchu42,hugo111,lining23666,tang-haojie,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
tangqunzhang成员
13 天前 评论:

compile

likedislike
Ttangqunzhang成员
13 天前 预合并成功(commit_id: 1cc5ecead0e1f933a12ad8655358e3881318579f)
CANN-robot
CANN-robot成员
13 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
13 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
13 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
13 天前 添加了label:ci-pipeline-passed
Ttangqunzhang成员
12 天前 修改标题为 “refactor: 删除 ge_rts_decl.h 中不再使用的日落接口”,原标题为“refactor: 删除 ge_rts_decl.h 中未使用的日落接口”
tang-haojie成员
12 天前 评论:

/lgtm

likedislike
tang-haojie成员
12 天前 评论:

【MR评价】评价分数:3,
评价意见:
编码规范遵守度:无编码规范问题;
代码设计:不涉及代码设计;
DT质量:基本逻辑DT测试覆盖充足;

likedislike
yuht9成员
12 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
12 天前 添加了label:lgtm
yangyongqiang
yangyongqiang成员
12 天前 评论:

/approve

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