已合并
[v2.9.0] update dvm submodule and profiling headers #35625
[v2.9.0] update dvm submodule and profiling headers #35625
已合并
SorryNaCN创建于 5月14日
SorryNaCN
SorryNaCN成员
5月14日

【合入来源】

DVM reshape bug 修复;CANN 9.1 profiling 头文件路径/注册兼容问题修复。

【修改方案】

  1. 更新 third_party/dvm/dvm 子仓到 r2.9 最新提交,带入 DVM reshape 场景相关 bug 修复。
  2. 带入 DVM 对 CANN 9.1 的 profiling 兼容修复,适配新的 profiling/prof_api.h 头文件路径及注册逻辑。
  3. third_party/acl/inc/profiling 下新增必要的 ACL profiling 头文件声明,仅保留当前 DVM/MLIR profiling 编译需要的 API、类型和宏定义。
  4. 删除旧 third_party/acl/inc/experiment/msprof/toolchain 头文件,避免继续依赖 experiment 路径。
  5. 将 MLIR launcher 中 msprof include 切换到 third_party/acl/inc/profiling 路径。

【资料变更】

不涉及。

【接口变更】

不涉及客户面可见接口变更。仅涉及仓内第三方 ACL profiling 头文件路径适配,满足 DVM 子仓使用 #include "profiling/prof_api.h" 的编译需求。

【功能验证】

  1. 已确认仓内无旧 experiment/msprof/toolchain 引用残留。
  2. 已通过最小 C++ include 编译检查,覆盖当前使用到的 Msprof API/类型声明。
  3. git diff --check 通过。

【CheckList】

PR提交人对以下CheckList自检项进行全量自检,自检通过或不涉及,均修改 [ ] 为 [x]

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 SorryNaCN 的贡献)
SorryNaCNSorryNaCN成员
5月14日 创建了 pull request,commit 91eddee0
ascend-robot
ascend-robot成员
5月14日 评论:

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/pytorch chujinjin, luochao60 (2/2) chujinjin (1/1)
torch_npu/_inductor chujinjin, luochao60 (2/2) chujinjin (1/1)
torch_npu/csrc/inductor luochao60, chujinjin (2/2) chujinjin (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
5月14日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
5月14日 评论:

当前仓库存在以下 保护分支

Protected Branch Version Release
master
v2.7.1
v2.11.0
v2.10.0
v2.9.0
v2.12.0

评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支(创建同步 PR):
a) 如果当前 PR 是 Open 状态,同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged,将立即执行同步操作

注意:

  1. /sync 命令可以指定同步到多个分支,仅最后一个 /sync 命令生效
  2. 如果创建的同步 PR 不正确,可通过向同步 PR 的源分支提交轻量级 PR 完善,或使用 /close 命令关闭
likedislike
SorryNaCNSorryNaCN成员
5月14日 修改了pull request 的描述
此处折叠了58条消息 查看更多
ascend-robotascend-robot成员
5月15日 添加了label:approvedlgtm
ascend-robot
ascend-robot成员
5月15日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: chujinjin.
Reviewers who wrote a comment of /lgtm are: chujinjin, luochao60.

likedislike
ascend-robotascend-robot成员
5月15日 合入了pull request
SorryNaCNSorryNaCN成员
5月15日 修改了pull request 的描述
ascend-robot
ascend-robot成员
5月15日 评论:
流水线 pytorch_gitcode_PR_multiVersion#9164 [ commitID:b9e8b9bc ] 已完成
likedislike