Star649
434
代码
代码
Issues31
Pull Requests109
Actions
项目讨论
Wiki
分析
Star649
434
已合并
refactor: 清理 dflow 中废弃函数和未使用常量 #3891
lining23666创建于 7月6日
refactor: 清理 dflow 中废弃函数和未使用常量 #3891
已合并
lining23666创建于 7月6日
从lining23666/geln_0224合入到cann/gedevelop
L
lining23666成员
7月6日

Pull Request

描述

清理 dflow 目录下经全仓库交叉引用分析确认的废弃代码：

删除 3 个全仓库无调用点的函数及其声明：

  • FlowModelOmSaver::SaveFlowModelToDataBuffer（flow_model_om_saver.cc）
  • FlowModelCache::UpdateFlowModelCache（flow_model_cache.cc，含关联常量 kMaxUpdateCacheThreadPoolSize）
  • AbnormalStatusHandler::FindAbnormalDevice（abnormal_status_handler.cc，实际调用的是 FindAbnormalDeviceOnServer）

删除 10 个全仓库零引用的常量：

  • kMaxErrstrLen、kUdfOpTypeFlowFunc、kDumpSliceSize、kBufferCount
  • kMbufHeadMaxSize、kNpuMaxGroupMemSize、kInvalidRankId、kCoreNumPerDevice
  • kConfigFileName、kHelperResFilePath

变更类型

关联的Issue

如何测试

  1. 增量编译 flow_graph target 通过（0 error）
  2. 增量编译 ut_libge_helper_utest target 通过（0 error）
  3. 所有废弃项均经全仓库交叉引用搜索确认（排除虚函数、pybind注册、测试桩等）

核对清单

其他信息

共 14 个文件，删除 105 行废弃代码。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 lining23666 的贡献）
Llining23666成员
7月6日 创建了 pull request，commit 7c75e27c
atomgit-bot
atomgit-bot
7月6日 评论：

变更摘要

本次 PR 对 dflow 目录进行了一次废弃代码清理，基于全仓库交叉引用分析，删除了 3 个无调用点的函数及其声明，以及十余个零引用的常量。涉及的模块包括模型保存 (FlowModelOmSaver)、模型缓存 (FlowModelCache)、异常状态处理 (AbnormalStatusHandler)，以及部署器、内存组、UDF dump 等多个子模块。所有删除项均经编译验证通过，属于纯重构，不影响现有功能。

主要改动

  • 删除 FlowModelOmSaver::SaveFlowModelToDataBuffer 函数：该函数在 flow_model_om_saver.cc 中定义、flow_model_om_saver.h 中声明，全仓库无调用点，一并移除。

  • 删除 FlowModelCache::UpdateFlowModelCache 函数及关联常量 kMaxUpdateCacheThreadPoolSize：函数位于 flow_model_cache.cc，负责将模型序列化后写回缓存文件，经交叉引用确认无调用点，连同其使用的线程池大小常量一起删除。

  • 删除 AbnormalStatusHandler::FindAbnormalDevice 函数：该函数在 abnormal_status_handler.cc 中实现（约 48 行），用于比较新旧部署配置来判定设备异常状态，实际调用方使用的是 FindAbnormalDeviceOnServer，因此移除。

  • 清理 12 个全仓库零引用的常量：分布在多个文件中，包括 kMaxErrstrLen、kUdfOpTypeFlowFunc、kDumpSliceSize、kBufferCount、kMbufHeadMaxSize、kNpuMaxGroupMemSize、kInvalidRankId、kCoreNumPerDevice、kConfigFileName、kHelperResFilePath、FLOW_FUNC_ERR_QUEUE_FULL 和 FLOW_FUNC_DISCARD_INPUT_ERROR，均已从各自所在文件中移除。

likedislike
atomgit-bot
atomgit-bot
7月6日 评论：

代码审查

审查总结

本次审查共检查 14 个变更文件，所有变更均为删除全仓库无引用的废弃函数和常量。

审查结果

优先级 数量
P0 0
P1 0
P2 0
P3 1

各文件审查结论

文件 结论
dflow/base/model/flow_model_om_saver.cc 无问题 — 删除 SaveFlowModelToDataBuffer 函数，已确认无调用点
dflow/base/model/flow_model_om_saver.h 无问题 — 同步删除函数声明
dflow/compiler/model/flow_model_cache.cc P3：删除 UpdateFlowModelCache 后遗留两处废弃 include（thread_pool.h、file_saver.h）
dflow/compiler/model/flow_model_cache.h 无问题 — 同步删除 UpdateFlowModelCache 声明
dflow/compiler/pne/udf/udf_model_builder.cc 无问题 — 删除常量 kUdfOpTypeFlowFunc，已确认无引用
dflow/deployer/common/config/configurations.cc 无问题 — 删除常量 kConfigFileName、kHelperResFilePath
dflow/deployer/common/data_flow/queue/heterogeneous_exchange_service.cc 无问题 — 删除常量 kBufferCount、kMbufHeadMaxSize
dflow/deployer/common/mem_grp/memory_group_manager.cc 无问题 — 删除常量 kNpuMaxGroupMemSize
dflow/deployer/deploy/abnormal_status_handler/abnormal_status_handler.cc 无问题 — 删除 FindAbnormalDevice 函数（仅日志字符串中含其名称，非代码引用）
dflow/deployer/deploy/abnormal_status_handler/abnormal_status_handler.h 无问题 — 同步删除函数声明
dflow/deployer/deploy/deployer/deployer.cc 无问题 — 删除常量 kInvalidRankId
dflow/deployer/deploy/deployer/deployer_var_manager.cc 无问题 — 删除常量 kCoreNumPerDevice
dflow/udf/toolchain/dump/udf_dump_task_device.cpp 无问题 — 删除常量 kDumpSliceSize
dflow/udf/toolchain/dump/udf_dump_task_host.cpp 无问题 — 删除常量 kMaxErrstrLen

总体评估

本次变更是低风险的代码清理重构。所有删除的函数和常量均已经过全仓库交叉引用验证，不存在未清理的残留引用或调用点。唯一的小瑕疵是 flow_model_cache.cc 中遗留了两处随函数删除而变为无用的 include，建议一并清理。整体风险：低。

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

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

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, xchu42, stevenaw0 (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

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

likedislike
CANN-robotCANN-robot成员
7月6日 将yangyongqiang0606,kobemini,tangqunzhang,xchu42,hugo111,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
7月6日 将yangyongqiang0606,kobemini,tangqunzhang,xchu42,hugo111,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
Llining23666成员
7月6日 update merge request[project id: 8824148, iid: 3891, commit_id: 1eaede8fcd5a69a060594bddf3b6627ed29ae10a] virtual merging success
Llining23666成员
7月6日 强制推送  1 个提交:455b1864-refactor: 清理 dflow 中废弃函数和未使用常量
Llining23666成员
7月6日 update merge request[project id: 8824148, iid: 3891, commit_id: c4cb9d0c626026d94af0e04818968e6a22303b5c] virtual merging success
Llining23666成员
7月6日 修改了pull request 的描述
L
lining23666成员
7月6日 评论：

compile

likedislike
Llining23666成员
7月6日 update merge request[project id: 8824148, iid: 3891, commit_id: 79acc6c1d4946e3be0aed345ce74cb72f5316d20] virtual merging success
CANN-robotCANN-robot成员
7月6日 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
7月6日 评论：

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

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

[2026-07-06 20:15:06]    CI执行结束

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

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

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

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

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

任务名称状态日志下载链接
codecheck_Pr ✅ SUCCESS
codecheck ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
codecheck_dt ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-07-06 20:13:07]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月6日 添加了label：api-check-pass
CANN-robotCANN-robot成员
7月6日 删除了label：api-check-pass
CANN-robotCANN-robot成员
7月6日 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
7月6日 添加了label：ci-pipeline-passed
xchu42
xchu42成员
7月6日 评论：

/lgtm

likedislike
黄桂军
黄桂军成员
7月6日 评论：

/lgtm

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

/approve

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