已合并
refactor: 将AscendC::MicroAPI替换为AscendC::Reg #1175
liu-wei创建于 7月27日
refactor: 将AscendC::MicroAPI替换为AscendC::Reg #1175
已合并
liu-wei创建于 7月27日
liu-wei
liu-wei成员
7月27日

描述

本 PR 将 ops-cv Regbase kernel 中的 AscendC::MicroAPI 命名空间统一替换为 AscendC::Reg,用于适配 Ascend C Reg API 命名空间调整。

主要改动:

  1. 更新 nms_with_maskresize_bilinear_v2resize_bilinear_v2_gradresize_nearest_neighbor_v2 的 arch35 Regbase kernel 代码。
  2. AscendC::MicroAPI::XMicroAPI::Xusing AscendC::MicroAPI::X 分别替换为 AscendC::Reg::XReg::Xusing AscendC::Reg::X
  3. 同步更新 docs/zh/develop/cross_platform_migration_guide.md 中 Regbase 编程范式说明和示例代码。

本次变更仅调整命名空间引用,不修改算子接口、tiling 数据结构、计算逻辑和构建脚本。

关联的Issue

#675

测试

文档更新

已更新 docs/zh/develop/cross_platform_migration_guide.md,将 Regbase 小节中的 MicroAPI 表述和代码示例同步调整为 AscendC::Reg / Reg

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 liu-wei 的贡献)
liu-weiliu-wei成员
7月27日 创建了 pull request,commit aa665e65
atomgit-bot
atomgit-bot
7月27日 评论:

变更摘要

本次变更是一个纯粹的命名空间重构:将 AscendC 算子内核代码中所有 AscendC::MicroAPI 命名空间下的类型、函数和常量引用,统一替换为新的 AscendC::Reg 命名空间。涉及 10 个文件,涵盖 NMSWithMask、ResizeBilinearV2、ResizeBilinearV2Grad 和 ResizeNearestNeighborV2 四个算子模块的 arch35 内核实现。所有替换均为等量替换(485 行新增,485 行删除),对外部接口和行为无任何影响。

主要改动

  • 命名空间从 MicroAPI 迁移到 Reg:所有文件中的 AscendC::MicroAPI:: 前缀(包括 using 声明中的 AscendC::MicroAPI::RegTensor 等)统一替换为 AscendC::Reg::,例如 using AscendC::Reg::RegTensorusing AscendC::Reg::AddrReg 等。

  • 核心寄存器类型和运算 API 前缀更新RegTensorMaskRegUnalignRegAddrReg 等寄存器类型,以及 DataCopyDuplicateSubMulAddMulsMinMaxCompareCastSelectPackUnPackMaskAndMaskNotMaskPackMaskDeInterleaveCreateMaskUpdateMaskCreateAddrRegLocalMemBarDataCopyUnAlignPost 等所有运算和工具函数的调用,均从 MicroAPI:: 改为 Reg::

  • 枚举和配置常量前缀更新CastTraitRegLayoutSatModeMaskMergeModeMaskPatternHighLowPartLoadDistStoreDistMaskDistPostLiteralMemType 等配置枚举和常量,全部从 MicroAPI:: 改为 Reg::,例如 Reg::CastTraitReg::RegLayout::ZEROReg::MaskPattern::ALL 等。

  • 注释中的术语同步更新:部分文件(如 resize_bilinear_v2_nc.hresize_bilinear_v2_grad_c_parallel.h)的代码注释也同步将 MicroAPI 改为 Reg,保持文档一致性。

likedislike
不准确?
atomgit-bot
atomgit-bot
7月27日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
7月27日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7月27日 评论:

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
image/resize_bilinear_v2 张磊, 王林木 (2/2) 张磊, 王林木 (2/1)
image/resize_bilinear_v2_grad 张磊, 王林木 (2/2) 张磊, 王林木 (2/1)
image/resize_nearest_neighbor_v2 王林木, 查建青, 张磊 (3/2) 王林木, 查建青 (2/1)
repo-cann/ops-cv 张磊, 王林木 (2/2) 张磊, 王林木 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了112条消息 查看更多
llimwang成员
7月31日 评论:

/approve

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

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

likedislike
CANN-robotCANN-robot成员
7月31日 添加了label:lgtm
CANN-robotCANN-robot成员
7月31日 关闭了关联的issue
CANN-robotCANN-robot成员
7月31日 合入了pull request