已合并
[HFusion] Fix generalization test missing pre-pass #2159
[HFusion] Fix generalization test missing pre-pass #2159
已合并
hujiajun创建于 7月31日
hujiajun成员
7月31日

描述 Description

Add --hfusion-pre-vectorization-fusion to the RUN line so named linalg ops are generalized to linalg.generic before AV2 processes them.

类型 Category

Checklist

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 hujiajun 的贡献)
Hhujiajun成员
7月31日 创建了 pull request,commit 44608012
atomgit-bot
atomgit-bot
7月31日 评论:

变更摘要

该 PR 用于修复 HFusion 泛化测试缺少预处理 pass 的问题:在测试文件 auto-vectorize-v2-generalization.mlir 的 RUN 命令行中、--hfusion-auto-vectorize-v2 之前新增 --hfusion-pre-vectorization-fusion,使命名 linalg 算子能在 AV2 处理前先被泛化为 linalg.generic,从而保证泛化测试按预期执行。

主要改动

  • 新增预处理 pass 调用:在 RUN 行中于 --hfusion-auto-vectorize-v2 前加入 --hfusion-pre-vectorization-fusion,确保命名 linalg 算子先泛化为 linalg.generic 再进入后续向量化流程。
  • 仅涉及测试用例文件:改动只发生在 bishengir/test/Dialect/HFusion/AutoVectorize/auto-vectorize-v2-generalization.mlir 的 RUN 行,属单行增减(+1 -1),不涉及生产代码逻辑变更。
likedislike
ascend-robot
ascend-robot成员
7月31日 评论:

libing-pipeline创建中,请等待……

likedislike
atomgit-bot
atomgit-bot
7月31日 评论:

代码审查

✅ 未发现问题

likedislike
ascend-robotascend-robot成员
7月31日 添加了label:ascend-cla/yes
ascend-robotascend-robot成员
7月31日 添加了label:SC-RUNNING
ascend-robot
ascend-robot成员
7月31日 评论:

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-Ascend/AscendNPU-IR 吴凌飞, NoBBAI (2/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

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

likedislike
ascend-robotascend-robot成员
7月31日 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
7月31日 评论:

✅ The pipeline is running (pipeline number: 1925), Task URL link, please wait a moment ...

likedislike
ascend-robot
ascend-robot成员
7月31日 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
7月31日 添加了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
7月31日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
7月31日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
7月31日 评论:
阶段 任务名 状态 详情
恶意代码检查 Antipoison >>>
流水线 PR-pipeline_ascendnpu-ir-A5 >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员[Jacky_XH, xiechenye]评论有效
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
ascend-robot
ascend-robot成员
7月31日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
7月31日 删除了label:docs-ci-pipeline-running
此处折叠了5条事件消息 查看更多
PpCSICompileDV成员
8月1日 通过测试
hujiajun成员
8月2日 评论:

coverage

likedislike
ascend-robotascend-robot成员
8月2日 添加了label:LCOV-running
ascend-robot
ascend-robot成员
8月2日 评论:

✅ The pipeline is running (pipeline number: 384), Task URL link, please wait a moment ...

likedislike
ascend-robotascend-robot成员
8月2日 删除了label:LCOV-running
ascend-robotascend-robot成员
8月2日 添加了label:LCOV-failure
ascend-robot
ascend-robot成员
8月2日 评论:
流水线 Ascend_npu_ir_coverage#384 运行失败
阶段 任务名 状态 详情
编译构建 Build构建 >>>
流水线 Ascend_npu_ir_coverage >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • coverage : 运行流水线
likedislike
lingfeiwu
lingfeiwu成员
8月3日 评论:

/lgtm
/approve

likedislike
ascend-robotascend-robot成员
8月3日 添加了label:approved
lingfeiwu
lingfeiwu成员8月3日进行代码检视1
bishengir/test/Dialect/HFusion/AutoVectorize/auto-vectorize-v2-generalization.mlir
@@ -1,0 +1,1 @@
1-// RUN: bishengir-opt %s --hfusion-auto-vectorize-v2 -outline-vector-function -split-input-file | FileCheck %s
1+// RUN: bishengir-opt %s --hfusion-pre-vectorization-fusion --hfusion-auto-vectorize-v2 -outline-vector-function -split-input-file | FileCheck %s
lingfeiwu
lingfeiwu8月3日评论:

再 auto-vectorize-v2 之前进行 hfusion-pre-vectorization-fusion 确保 pass 执行前的 ir 符合 pass 预期

likedislike
lingfeiwulingfeiwu成员
8月3日 解决了最后一个问题
hujiajun成员
8月3日 评论:

coverage

likedislike
ascend-robotascend-robot成员
8月3日 删除了label:LCOV-failure
ascend-robotascend-robot成员
8月3日 添加了label:LCOV-running
ascend-robot
ascend-robot成员
8月3日 评论:

✅ The pipeline is running (pipeline number: 397), Task URL link, please wait a moment ...

likedislike
ascend-robotascend-robot成员
29 天前 删除了label:LCOV-running
ascend-robotascend-robot成员
29 天前 添加了label:LCOV-failure
ascend-robot
ascend-robot成员
29 天前 评论:
流水线 Ascend_npu_ir_coverage#397 运行失败
阶段 任务名 状态 详情
编译构建 Build构建 >>>
流水线 Ascend_npu_ir_coverage >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • coverage : 运行流水线
likedislike
NoBBAI成员
29 天前 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
29 天前 添加了label:lgtm
ascend-robotascend-robot成员
29 天前 合入了pull request,合并节点 SHA:9865f42094337e9ec3c0bf985f87c5867920f7f4