已合并
refactor: 清理block_mem_zero_copy未使用变量并优化注释 #4522
refactor: 清理block_mem_zero_copy未使用变量并优化注释 #4522
已合并
tangqunzhang创建于 8 天前
tangqunzhang成员
8 天前

Pull Request

描述

清理 block_mem_zero_copy.cc 中未使用的匿名命名空间和 kOffline 常量,并优化 HCOM 算子地址刷新相关注释的表述,使逻辑更清晰。

变更类型

关联的Issue

如何测试

  1. 编译 GE 项目确认无编译错误
  2. 运行相关 UT 确认无功能回归

核对清单

其他信息

变更内容:

  1. 移除未使用的匿名命名空间 kOffline = "offline" 常量
  2. 优化注释:将分散的注释合并为更连贯的表述,明确说明统一使用 IsHcomNodeNotSupportAddrRefresh 结果(不可刷新)
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 tangqunzhang 的贡献)
Ttangqunzhang成员
8 天前 创建了 pull request,commit 1d877ab8
atomgit-bot
atomgit-bot
8 天前 评论:

变更摘要

该 PR 是对 compiler/graph/build/memory/block_mem_zero_copy.cc 的代码清理与注释优化(重构 + 代码风格更新),不涉及功能或逻辑变更。具体包括:移除未使用的匿名命名空间及其中的 kOffline = "offline" 常量,并将 HCOM 算子地址刷新相关分散注释合并为更连贯的表述,明确说明在 IsNodeSupportZeroCopy 中统一使用 IsHcomNodeNotSupportAddrRefresh 的结果(不可刷新)并保持原有处理方式。

主要改动

  • 移除未使用常量: 删除文件中的匿名命名空间及 kOffline = "offline" 常量定义,消除未使用变量。
  • 合并并优化注释: 将 IsNodeSupportZeroCopy 中关于 IsAddressRefreshable 在动态 shape 静态子图场景下对 hccl 算子判断结果不一致、以及静态子图支持刷新带来性能劣化的分散注释合并为连贯表述。
  • 明确处理口径: 注释中强调仍保持原有处理,统一使用 IsHcomNodeNotSupportAddrRefresh 的结果(不可刷新)作为判断依据,不改变实际逻辑行为。
likedislike
atomgit-bot
atomgit-bot
8 天前 评论:

🤖 本次改动均为数据 / 生成 / 二进制文件(如测试用例、锁文件、媒体资源),没有可审查的代码逻辑,已跳过 AI 代码审查。

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

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 张德鹏, kobemini, Mexyy (3/2) 张德鹏 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了47条消息 查看更多
GengChao
GengChao成员
8 天前 评论:

/lgtm

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

/approve

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