已合并
【PR】: feat:增加report match接口 #4360
hugo111创建于 25 天前
【PR】: feat:增加report match接口 #4360
已合并
hugo111创建于 25 天前
hugo111成员
25 天前

Pull Request

描述

20260723
新增reportMatch接口用于融合pass上报匹配结构命中的维测信息

变更类型

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

关联的Issue

如何测试

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

核对清单

其他信息

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

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 hugo111 的贡献)
Hhugo111成员
25 天前 添加了label:enhancement
Hhugo111成员
25 天前 创建了 pull request,commit 595a7864
atomgit-bot
atomgit-bot
25 天前 评论:

变更摘要

本次 PR 主要为融合图模式检查工具新增结构匹配上报能力:在 GraphFuseInspectorUtils 中新增 ReportMatch 接口,用于在发现目标子图结构后记录一次匹配(match_time),并将原 ReportFuse 的统计逻辑调整为只记录生效次数(effect_time),从而支持统计结构匹配命中率。同时补充了示例用法与单元测试。

主要改动

  • 新增 ReportMatch 接口: 在 GraphFuseInspectorUtils 中新增静态方法 ReportMatch,将匹配命中的节点列表与 CustomPassContext 传入,通过 FusionUtils::RecordFusionStatistic1, 0 的参数累加 match_time、不改变 effect_time
  • 调整 ReportFuse 统计逻辑: 将 ReportFuse 中对 RecordFusionStatistic 的调用参数由 1, 1 改为 0, 1,使其仅记录生效次数,不再重复计入匹配次数。
  • 示例 pass 接入匹配上报: 在 modify_conv_data_format_pass.cppRemoveTransposeAndRelink 中,于 CanFuse 判断前调用 ReportMatch,无论融合条件是否通过均计入匹配次数。
  • 更新接口文档注释: 在 graph_fuse_inspector_utils.h 中补充 ReportMatch 的声明与说明,并同步更新 ReportFuse 的行为注释为“记录生效次数”。
  • 补充单元测试: 在 graph_fuse_inspector_utils_unittest.cc 中新增对 ReportMatch 空节点、非法 GNode、空 pass 名、跨图节点等失败场景及成功场景的测试,并校验 ReportMatch 仅累加 match_timeReportFuse 仅累加 effect_time
likedislike
atomgit-bot
atomgit-bot
25 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
25 天前 添加了label:cann-cla/yes
此处折叠了192条消息 查看更多
张德鹏成员
10 天前 评论:

/approve

likedislike
Hhugo111成员
10 天前 修改了pull request 的描述
shengnan成员
10 天前 评论:

/approve

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