已合并
math算子目录与代码中误用的arch32统一改为arch22 #4873
clinglai0517创建于 21 天前
math算子目录与代码中误用的arch32统一改为arch22 #4873
已合并
clinglai0517创建于 21 天前
clinglai0517成员
21 天前

描述

将math目录下误用arch32的算子目录、文件名及代码引用统一修正为
arch22(与cdist_grad、segsum等算子的arch22约定保持一致):

  1. 目录重命名(6处):

    • math/bessel_i1e/op_host/arch32 -> arch22
    • math/is_finite/op_host/arch32 -> arch22
    • math/histogram_v2/tests/ut/op_host/arch32 -> arch22
  2. 文件重命名(3个):bessel_i1e_tiling_arch32.cpp、
    is_finite_tiling_arch32.h/.cpp -> 对应arch22文件名

  3. 内容替换(42个文件):

    • 约30个算子的CMakeLists.txt中SUPPORT_TILING_DIR的"arch32" -> "arch22"
    • include路径、is_finite的结构体/函数名/include guard
    • 相关注释
  4. 同步更新classify_rule.yaml中4处math路径条目。

【影响范围】

  • 涉及算子:bessel_i1e、is_finite、
    histogram_v2,以及CMakeLists中引用arch32的约30个math算子
  • 仅改动命名与引用,未修改任何算子逻辑
  • 纯重命名,git可完整追踪(16个文件100%相似度重命名)

关联的Issue

Issue #2749

测试

  • 算子编译及ut通过
  • math目录下rg检索arch32(含Arch32/ARCH32)无残留

文档更新

不涉及

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 clinglai0517 的贡献)
Cclinglai0517成员
21 天前 创建了 pull request,commit 462b064d
Cclinglai0517成员
21 天前 关联了issue:[Bug-Report|缺陷反馈]: A2A3相关算子对应的架构缩写使用arch22,当前部分误用了arch32
atomgit-bot
atomgit-bot
21 天前 评论:

变更摘要

本 PR 为修复误用架构名称的问题,将 math 目录下算子中误写的 arch32 统一修正为 arch22(与 cdist_gradsegsum 等算子的 arch22 约定保持一致),涉及 bessel_i1egrouped_bias_add_gradis_finite 等算子的目录名、文件名、头文件包含路径、代码符号及注释。改动包括目录/文件的批量重命名(arch32/arch22/)、构建配置 classify_rule.yaml 中的路径更新,以及相关 C++ 代码中 arch32 字符串、类型名与函数名的同步替换。

主要改动

  • 构建规则路径修正classify_rule.yamlbessel_i1e/op_hostgrouped_bias_add_grad/op_hostgrouped_bias_add_grad/op_kernelis_finite/op_host 的归档路径由 arch32/ 全部改为 arch22/,确保构建系统能匹配到重命名后的目录。
  • 目录与文件批量重命名:将 bessel_i1egrouped_bias_add_grad(含 op_hostop_kernel、UT 目录)、is_finite(含 op_host)下的 arch32 目录及相关文件名(如 is_finite_tiling_arch32.cpp/his_finite_tiling_arch22.cpp/h)统一重命名为 arch22
  • 头文件包含路径同步更新grouped_bias_add_grad.cpp 中的 #include "arch32/grouped_bias_add_grad_*.h" 改为 arch22 版本;test_grouped_bias_add_grad.cpptest_grouped_bias_add_grad_infershape.cpptest_is_finite.cpp 中引用 op_host/arch32/... 头文件的路径一并更新为 op_host/arch22/...
  • 代码符号重命名is_finite 的 tiling 实现中将函数 TilingPrepare4IsFiniteArch32IsFiniteTilingArch32 及类型 IsFiniteCompileInfoArch32 统一更名为 Arch22 版本,头文件宏定义 IS_FINITE_TILING_ARCH32_H 同步改为 IS_FINITE_TILING_ARCH22_H,UT 中对该编译信息结构的引用也随之更新。
  • 注释文案修正histogram_v2 的 UT 相关文件(arch22/test_histogram_v2_tiling.cpparch35/test_histogram_v2_tiling.cpphistogram_v2_tiling_ut_util.h)中的 arch32 注释文案统一改为 arch22,与重命名后的目录保持一致。
likedislike
不准确?
atomgit-bot
atomgit-bot
21 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
21 天前 添加了label:cann-cla/yes
此处折叠了144条消息 查看更多
Cclinglai0517成员
19 天前 修改了pull request 的描述
yue-ma成员
19 天前 评论:

/lgtm
/approve

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