Star1030
2427
代码
代码
Issues153
Pull Requests339
Actions
项目讨论
Wiki
分析
Star1030
2427
已合并
添加mhcsinkhorn tiling编译依赖，解决common编译依赖问题 #9030
fengjiawei1创建于 16 天前
添加mhcsinkhorn tiling编译依赖，解决common编译依赖问题 #9030
已合并
fengjiawei1创建于 16 天前
从fengjiawei1/ops-transformermhc_oom合入到cann/ops-transformermaster
F
fengjiawei1成员
16 天前

描述

添加mhcsinkhorn tiling编译依赖, 解决出包编译问题

关联的Issue

https://gitcode.com/cann/ops-transformer/issues/3900

测试

文档更新

类型标签

  • 🐛 Bug修复
  • ✨ 新特性
  • ⚡ 性能优化
  • ♻️ 重构
  • 🧪 测试
  • 📦 构建/CI
  • 🔧 配置变更
  • 📝 文档更新
  • ⬆️ 依赖升级
  • 🔒 安全修复
  • 🧹 代码清理
  • ❓ 其他，请描述：
likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 fengjiawei1 的贡献）
Ffengjiawei1成员
16 天前 创建了 pull request，commit 8167449e
atomgit-bot
atomgit-bot
16 天前 评论：

变更摘要

此 PR 主要对 mhc_sinkhorn.h 中的 MhcSinkhornSimd 模板类进行了代码简化和清理，移除了多个函数中冗余的掩码寄存器（maskRegFresh）和 MaskAnd 合并操作，直接使用更新后的 maskRegCalc 作为计算掩码。同时调整了非输出场景下 sumRowQue_ 的缓冲区初始化大小为零，并清理了部分未使用变量及文件末尾格式。

主要改动

  • 移除冗余的掩码合并逻辑：在 CalcSoftmax、CalcColNorm、CalcRowNorm 和 ScatterOutFromNorm 四个函数中，删除了 maskRegFresh 声明及 MicroAPI::MaskAnd(maskRegCalc, maskRegCalc, maskRegFresh, maskReg) 调用，改为直接用 maskRegCalc = MicroAPI::UpdateMask<uint32_t>(handleSize) 作为计算掩码，简化掩码处理流程。
  • Init 中非输出场景缓冲区大小归零：当 OUT_FLAG 为 false 时，将 pipe_.InitBuffer(sumRowQue_, DOUBLE_BUFFER, ...) 的缓冲区大小从 blockDataNum_ * sizeof(float) 改为 0 * sizeof(float)。
  • 清理 CalcColNorm 中未使用的 maskReg 创建：删除了 maskReg = MicroAPI::CreateMask<uint32_t>() 调用，因为该函数内不再使用该初始掩码。
  • 清理 ScatterOutFromNorm 中未使用的 maskRegFresh 声明：移除了一处未被使用的 MaskReg maskRegFresh; 局部变量声明。
  • 文件末尾格式修复：在 #endif 后补充了缺失的换行符。
likedislike
不准确？
CANN-robotCANN-robot成员
16 天前 添加了label：cann-cla/yes
CANN-robot
CANN-robot成员
16 天前 评论：

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
sig-ops-transformer ✅ 俞郑中, 黄俊健 (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

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

likedislike
CANN-robotCANN-robot成员
16 天前 将monologue815,jiang-lirui,Allan_Yu,liudan12,mabing1118设为评审人
此处折叠了7条事件消息 查看更多
Ffengjiawei1成员
15 天前 修改标题为 “添加mhcsinkhorn tiling编译依赖”，原标题为“mhc test oom”
F
fengjiawei1成员
15 天前 评论：

compile

likedislike
Ffengjiawei1成员
15 天前 update merge request[project id: 7673863, iid: 9030, commit_id: 950a8a7cccb5037c3ade2ff41edcd6b41489b3ea] virtual merging success
CANN-robotCANN-robot成员
15 天前 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
15 天前 评论：

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Pre ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_950_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental_950_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_classify ✅ SUCCESS >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station_9030_ubuntu24 ✅ SUCCESS >>>>> >>>>>

[2026-07-22 17:42:30]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
15 天前 评论：

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_style ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_checkpr ✅ SUCCESS
codecheck_precommit ⚠️ WARNING >>>>> >>>>>

[2026-07-22 17:37:15]    CI执行结束

likedislike
Ffengjiawei1成员
15 天前 修改了pull request 的描述
Ffengjiawei1成员
15 天前 修改了pull request 的描述
Ffengjiawei1成员
15 天前 update merge request[project id: 7673863, iid: 9030, commit_id: 48910ec16238f8f277aba432e542b0b0e5d41ae8] virtual merging success
Ffengjiawei1成员
15 天前 强制推送  1 个提交:9a723c02-mhc catest
Ffengjiawei1成员
15 天前 update merge request[project id: 7673863, iid: 9030, commit_id: 2fe33a57a8b4be890653c9a95db13e1be6fa5c6d] virtual merging success
F
fengjiawei1成员
15 天前 评论：

compile

likedislike
Ffengjiawei1成员
15 天前 update merge request[project id: 7673863, iid: 9030, commit_id: a1d60686411d03cfd551a2a94ef0faa166703c4f] virtual merging success
CANN-robotCANN-robot成员
15 天前 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
15 天前 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
15 天前 评论：

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

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Pre ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_950_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental_950_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_classify ✅ SUCCESS >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station_9030_ubuntu24 ✅ SUCCESS >>>>> >>>>>
UT_Test ✅ SUCCESS
PreSmoke_A900 ✅ SUCCESS >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>
UT_Test_harmony-infer-chs-transformer ✅ SUCCESS
UT_Test_report_lcov ⚠️ WARNING >>>>>

[2026-07-22 17:58:17]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
15 天前 评论：

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_style ✅ SUCCESS >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_checkpr ✅ SUCCESS
codecheck_precommit ✅ SUCCESS >>>>>

[2026-07-22 17:45:56]    CI执行结束

likedislike
CANN-robotCANN-robot成员
15 天前 添加了label：api-check-pass
CANN-robotCANN-robot成员
15 天前 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
15 天前 添加了label：ci-pipeline-passed
Ffengjiawei1成员
15 天前 修改标题为 “添加mhcsinkhorn tiling编译依赖，解决common编译依赖问题”，原标题为“添加mhcsinkhorn tiling编译依赖”
M
monologue815成员
15 天前 评论：

/lgtm

likedislike
K
kevin_huang1234
15 天前 评论：

/lgtm
/approve

likedislike
CANN-robot
CANN-robot成员
15 天前 评论：

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
黄迪

likedislike
CANN-robot
CANN-robot成员
15 天前 评论：

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
黄迪

likedislike
Y
YuZhengzhong成员
15 天前 评论：

/approve
/lgtm

likedislike
CANN-robot
CANN-robot成员
15 天前 评论：

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
黄迪

likedislike
CANN-robotCANN-robot成员
15 天前 添加了label：lgtmapproved
CANN-robotCANN-robot成员
15 天前 关闭了关联的issue
CANN-robotCANN-robot成员
15 天前 合入了pull request