已合并
refactor: 迁移 arch35 算子 Reg API、修正拼写并规范化日志文案 #5135
esok11创建于 3 天前
refactor: 迁移 arch35 算子 Reg API、修正拼写并规范化日志文案 #5135
已合并
esok11创建于 3 天前
esok11成员
3 天前

描述

对 arch35 架构下多个算子进行 AscendC Reg API 迁移、命名拼写修正及日志文案规范化,共 46 个文件,涉及 conversion/ 与 math/ 下 15 个算子。

主要改动:

  • AscendC Reg API 迁移(arch35 kernel 代码,21 个 .h 文件):
    • 地址空间修饰符:__local_mem____ubuf__
    • Reg 数据搬运 API 细化:加载用 AscendC::Reg::LoadAlign(原 Reg::DataCopy 加载语义),存储用 AscendC::Reg::StoreAlign(原 Reg::DataCopy 存储语义)
    • 掩码操作 API 重命名:AscendC::Reg::MaskNotAscendC::Reg::NotAscendC::Reg::MaskAndAscendC::Reg::And
    • 移除冗余 using AscendC::Reg::DataCopy; 声明
  • 命名拼写修正(op_host/arch35,3 个文件):
    • 方法名 TilingForReleatedTransposeTilingForRelatedTranspose(Releated→Related)
    • 成员变量 isReleatedTranspsoe_isRelatedTranspose_(Releated→Related, Transpsoe→Transpose)
    • 同步修改 confusion_transpose_d、depth_to_space 中对该方法的调用
  • 日志清理(examples、op_host、op_api,22 个文件):
    • 失败路径日志级别 INFOERROR(Initialize ge failed、Run graph failed、Finalize ir graph session failed 等)
    • 文案语法修正:only supportonly supportsis: %s and %sare: %s and %sexpect %fexpected %fmultiples'smultiples'
    • tile 示例代码宏定义缩进格式化

改动原因

配合 AscendC Reg API 演进,将 arch35 算子迁移至新的地址空间修饰符与 LoadAlign/StoreAlign/Not/And API;修正 transpose 相关命名中的拼写错误;统一日志级别与文案规范,提升日志可读性与问题定位效率。

改动方法

  • kernel 侧:逐文件替换 __local_mem____ubuf__Reg::DataCopy(加载)→Reg::LoadAlignReg::DataCopy(存储)→Reg::StoreAlignReg::MaskNotReg::NotReg::MaskAndReg::And,不改变控制流。
  • host 侧:重命名 TilingForReleatedTransposeTilingForRelatedTransposeisReleatedTranspsoe_isRelatedTranspose_,同步所有调用点。
  • 日志:修改 printf/OP_LOGE/OP_LOGW/OP_LOGI 文案与级别字符串,不改变校验逻辑。

关联的Issue

  • #2916

测试

本 PR 为 API 迁移、命名修正与日志清理,通过已有测试用例(含 arch35 tiling UT 与 examples 二级冒烟)回归验证。

文档更新

本 PR 未包含文档更新。

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 esok11 的贡献)
Eesok11成员
3 天前 创建了 pull request,commit e97774ab
atomgit-bot
atomgit-bot
3 天前 评论:

变更摘要

本 PR 主要包含两类改动:一是对 AscendC 算子 kernel 的寄存器级(AscendC::Reg)与 ND DMA 相关 API 进行更名适配,涉及 broadcast_topad_v3pad_v3_gradpad_v3_grad_replicationtruncate_div 等算子的 op_kernel 代码;二是日志清理,包括示例程序中失败路径的日志级别从 INFO 修正为 ERROR、日志措辞与变量命名(如 findedfound)的规范化,以及 transpose tiling 接口的拼写修正。整体不改变业务逻辑,仅做接口适配与日志/命名清理。

主要改动

  • AscendC 寄存器级 API 更名:在 pad_v3/pad_v3_grad/pad_v3_grad_replication/truncate_div/broadcast_to 等 kernel 中,将 Reg::DataCopy 按读写语义拆分为 Reg::LoadAlign/Reg::StoreAlignDataCopyUnAlign* 更名为 LoadUnAlign*/StoreUnAlign*DataCopyGatherGatherDataCopyScatterScatterCompareScalarComparesCopyMoveMaskNot/MaskAndNot/AndMaskSel/GatherMaskSelect/Squeeze,并将非对齐辅助类型 UnalignReg 拆分为 UnalignRegForLoad/UnalignRegForStore
  • ND DMA 类型更名与指针标注调整broadcast_to kernel 中 MultiCopyConfigNdDmaConfigMultiCopyLoopInfoNdDmaLoopInfoMultiCopyParamsNdDmaParams;同时各 kernel 中 UB 内存指针标注由 __local_mem__ 统一改为 __ubuf__
  • Transpose tiling 接口拼写修正TransposeNddmaTilingTilingForReleatedTranspose 更名为 TilingForRelatedTranspose,成员 isReleatedTranspsoe_ 更名为 isRelatedTranspose_,并同步更新 transposespace_to_depthdepth_to_spaceconfusion_transpose_d 的 tiling 调用点。
  • 日志与命名清理:各 test_geir_*.cpp 示例中初始化/建图/运行/终结失败路径的日志级别由 INFO 改为 ERROR;日志措辞规范化(如 "is/are"、"expect"→"expected"、multiples'smultiples'、"Format only support ND"→"supports ND"),pad_v3pad_v3_grad tiling 中防死循环标志变量 finded 统一改为 found,并对 math/tile 示例代码做了格式化整理。
likedislike
atomgit-bot
atomgit-bot
3 天前 评论:

代码审查

✅ 未发现问题

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

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
3 天前 评论:

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
conversion/broadcast_to zhanw_coding, 汤平川 (2/2) zhanw_coding, 汤平川 (2/1)
conversion/transpose zhanw_coding, 汤平川 (2/2) zhanw_coding, 汤平川 (2/1)
math/erfc 汤平川, zhanw_coding (2/2) zhanw_coding, 汤平川 (2/1)
math/tile zhanw_coding, 汤平川 (2/2) 汤平川, zhanw_coding (2/1)
repo-cann/ops-math zhanw_coding, 汤平川 (2/2) zhanw_coding, 汤平川 (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
3 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为评审人
CANN-robotCANN-robot成员
3 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为审查人
esok11成员
3 天前 评论:

/compile

likedislike
Eesok11成员
3 天前 update merge request[project id: 7649531, iid: 5135, commit_id: d469624f21a454df942d515bc5ffa02a9f0ac792] virtual merging success
CANN-robotCANN-robot成员
3 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
3 天前 评论:

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

任务名称状态日志下载链接
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_A5_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_A5_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station_9030_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
UT_Test ✅ SUCCESS
UT_Test_experimental ✅ SUCCESS
UT_Test_kernel ✅ SUCCESS
PreSmoke_A900 ✅ SUCCESS >>>>>
API_Check ⚠️ WARNING >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>

[2026-08-27 20:21:06]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
3 天前 评论:

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

任务名称状态日志下载链接
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Pr ✅ SUCCESS
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_md ✅ SUCCESS

[2026-08-27 18:51:47]    CI执行结束

likedislike
Eesok11成员
3 天前 修改标题为 “refactor: 迁移 arch35 算子 Reg API、修正拼写并规范化日志文案”,原标题为“asc op api modify & log clean”
Eesok11成员
3 天前 修改了pull request 的描述
CANN-robotCANN-robot成员
3 天前 添加了label:api-check-failed
CANN-robotCANN-robot成员
3 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
3 天前 添加了label:ci-pipeline-passed
esok11成员
3 天前 评论:

/check-pr

likedislike
CANN-robot
CANN-robot成员
3 天前 评论:

The following labels are not ready.

lgtm: Please wait for reviewers to review the code.

approved: Please wait for committers to review the code.

likedislike
zhanw_coding
zhanw_coding成员
3 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
3 天前 添加了label:approved
TangPC
TangPC成员
3 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
3 天前 添加了label:lgtm
CANN-robotCANN-robot成员
3 天前 关闭了关联的issue
CANN-robotCANN-robot成员
3 天前 合入了pull request