Star659
446
代码
代码
Issues34
Pull Requests128
Actions
项目讨论
Wiki
分析
Star659
446
已合并
【fix】: 告警清理 #4048
ZhuXincheng创建于 7月17日
【fix】: 告警清理 #4048
已合并
ZhuXincheng创建于 7月17日
从SeasonChu/ge_7912CleanCode合入到cann/gedevelop
ZhuXincheng
ZhuXincheng成员
7月17日

Pull Request

描述

清理头文件中未用const修饰但是在实现时增加了const修饰的告警

变更类型

请选择本次引入的变更类型：

关联的Issue

如何测试

描述测试此变更的步骤和前提条件：
1.
2.

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 ZhuXincheng 的贡献）
ZhuXinchengZhuXincheng成员
7月17日 添加了label：enhancement
ZhuXinchengZhuXincheng成员
7月17日 创建了 pull request，commit 932b6fb2
atomgit-bot
atomgit-bot
7月17日 评论：

变更摘要

此 PR 旨在修复编译器告警：hybrid_model_rt_v2_executor.h 头文件中多个成员函数的参数声明缺少 const 修饰，而其对应的 .cpp 实现文件中已添加了 const，导致声明与定义不一致。变更统一在头文件声明中为这些参数补齐 const，消除告警。

主要改动

  • 为 size_t 类型参数统一添加 const：在 IsHostModelInput、SetInputOnHost、SetGeTensorRtInputPlacement、SetGeTensorRtInputData、PrepareGeTensorRtInput、PrepareRtInputFromInputData、CopyGertInputToDevice、PrepareRtInputFromGertTensor 等函数的 input_index 参数前增加 const 修饰（如 const size_t input_index），使头文件声明与实现一致。
  • 为 ge::Allocator * 类型参数添加顶层 const：在 PrepareRtInputFromInputData、CopyGertInputToDevice、PrepareRtInputFromGertTensor、PrepareInputDataRtInputs、PrepareGertRtInputs 等函数中将 ge::Allocator *allocator 改为 ge::Allocator *const allocator，确保指针本身不可变，与实现匹配。
  • 为 gert::Allocators * 类型参数添加顶层 const：将 ExecuteRtModel 的 gert::Allocators *allocators 改为 gert::Allocators *const allocators，使声明与定义一致。
likedislike
atomgit-bot
atomgit-bot
7月17日 评论：

代码审查

所有修改的行均已审查完毕。以下是最终分析结果。

审查结论摘要

变更文件： runtime/v1/hybrid/executor/hybrid_model_rt_v2_executor.h（已审查）

  • P0： 0 个
  • P1： 0 个
  • P2： 0 个
  • P3： 2 个

总体风险评估： 低风险。该 PR 为头文件中的函数参数补充 const 修饰，以消除与 .cc 实现之间的声明不一致告警。所有添加的 const 均为按值传递参数的顶层 const 或指针自身的顶层 const，不影响函数类型、链接或运行时行为。但该修复不完整——在同一声明行中遗漏了 addr、size 以及三个函数中的 cur_device_id 参数的 const 修饰，导致这些位置的告警仍未消除。

⚠️ 已识别出整体风险，但无法提取行内评论，请参考整体评估。

likedislike
CANN-robotCANN-robot成员
7月17日 添加了label：cann-cla/yes
CANN-robot
CANN-robot成员
7月17日 评论：

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
7月17日 评论：

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

PR Approval Progress

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

Module Approval Details

module lgtm status approve status
repo-cann/ge ✅ zhanj, kobemini, yangyongqiang0606 (3/2) ✅ zhanj (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
7月17日 将yangyongqiang0606,kobemini,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
7月17日 将yangyongqiang0606,kobemini,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
ZhuXincheng
ZhuXincheng成员
7月17日 评论：

compile

likedislike
ZhuXinchengZhuXincheng成员
7月17日 update merge request[project id: 8824148, iid: 4048, commit_id: 59c8fae2fe3afbf5173a756107c962e6bf329b85] virtual merging success
ZhuXinchengZhuXincheng成员
7月17日 修改了pull request 的描述
ZhuXinchengZhuXincheng成员
7月17日 修改了pull request 的描述
ZhuXinchengZhuXincheng成员
7月17日 修改了pull request 的描述
ZhuXinchengZhuXincheng成员
7月17日 修改了pull request 的描述
CANN-robotCANN-robot成员
7月17日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
7月17日 评论：

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

任务名称状态日志下载链接
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-07-17 16:21:54]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
7月17日 评论：

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

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

[2026-07-17 16:14:45]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
7月17日 评论：

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

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
codecheck ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ SUCCESS >>>>>
codecheck_precommit ❌ FAILED >>>>> >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
likedislike
CANN-robotCANN-robot成员
7月17日 添加了label：api-check-pass
此处折叠了5条事件消息 查看更多
ZhuXinchengZhuXincheng成员
7月17日 update merge request[project id: 8824148, iid: 4048, commit_id: 36291a85ff78a1203ad140304e565855fef5202b] virtual merging success
ZhuXincheng
ZhuXincheng成员
7月17日 评论：

compile

likedislike
ZhuXinchengZhuXincheng成员
7月17日 update merge request[project id: 8824148, iid: 4048, commit_id: f28a51f1074fd39b689614398294c313a3857100] virtual merging success
CANN-robotCANN-robot成员
7月17日 删除了label：ci-pipeline-failed
CANN-robotCANN-robot成员
7月17日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
7月17日 评论：

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

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

[2026-07-17 16:32:19]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
7月17日 评论：

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

任务名称状态日志下载链接
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-07-17 16:39:48]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
7月17日 评论：

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

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
codecheck ✅ 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-07-17 16:30:29]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月17日 添加了label：api-check-pass
CANN-robotCANN-robot成员
7月17日 删除了label：api-check-pass
CANN-robotCANN-robot成员
7月17日 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
7月17日 添加了label：ci-pipeline-passed
ZhuXinchengZhuXincheng成员
7月17日 修改了pull request 的描述
GengChao
GengChao成员
7月17日 评论：

/lgtm

likedislike
yangyongqiang
yangyongqiang成员
7月17日 评论：

/lgtm

likedislike
CANN-robotCANN-robot成员
7月17日 添加了label：lgtm
zhanj
zhanj成员
7月18日 评论：

/approve

likedislike
CANN-robotCANN-robot成员
7月18日 添加了label：approved
CANN-robotCANN-robot成员
7月18日 合入了pull request