已合并
refactor(segsum): arch35 内核头文件更名为 segsum_arch35.h,消除与 A2 同名 #5035
refactor(segsum): arch35 内核头文件更名为 segsum_arch35.h,消除与 A2 同名 #5035
已合并
zl_hw创建于 13 天前
zl_hw成员
13 天前

描述

math/segsum/op_kernel/arch35/segsum.h 更名为 segsum_arch35.h,并同步 arch35/segsum.cpp 的 include。

改动原因:该头文件与 A2 的 math/segsum/op_kernel/segsum.h 基名完全相同,两者都可能出现在内核编译的头文件搜索路径上,#include "segsum.h" 的落点依赖 include 顺序;检视与排障时也无法从文件名分辨代次。仓内 248 个含 op_kernel/arch35 的算子中仅 4 个存在这种同名(mul_addn、pad_v3_grad_replication、exp_segsum_grad、segsum),主流写法是 arch35 侧另起名(如 conversion/slice/op_kernel/arch35/slice_arch35.h);本算子 host 侧已经是 segsum_tiling_arch35.cpp/.h,本次把内核头文件对齐。

改动方法(2 个文件,各 1 行):

  • op_kernel/arch35/segsum.hop_kernel/arch35/segsum_arch35.h(git rename,相似度 99%,仅同步 \file 注释)
  • op_kernel/arch35/segsum.cpp#include "segsum.h"#include "segsum_arch35.h"

未改动项:

  • include guard 本就是 SEGSUM_ARCH35_H、命名空间本就是 SegsumArch35,无需调整
  • 入口源文件 arch35/segsum.cpp 不更名:.cpp 不被 include,由 CMakeLists.txtKERNEL_SRC arch35/segsum.cpp 按显式路径编译,segsum_def.cppExtendCfgInfo("opFile.value", "segsum") 也以算子名为准,同名不产生歧义
  • 内核入口函数名仍为 segsum;A2 侧 op_kernel/segsum.{cpp,h} 未改动;arch35/segsum_tiling_data.h 无同名冲突,未改动

影响面核查:全仓检索旧名引用,除本次改掉的一处外无其他引用;产物命名不受影响(<op>_apt 目录与 json 取自 OP_NAME 而非文件名,见 cmake/gen_ops_info.cmake);kernel UT 按 KERNEL_SRC 的值 glob 入口文件(cmake/ut.cmake),入口未更名故不受影响;打包用 find -maxdepth 1 -exec cp -r 保留 arch35/ 子目录结构。

关联的Issue

测试

重新出包:bash build.sh --pkg --soc=ascend950 --ops=segsum,编译无 error;出包内 op_impl/ai_core/tbe/custom_math_impl/ascendc/segsum/arch35/ 下已只有 segsum_arch35.h

真机 Ascend950PR(TTK kernel 模式,-d false -b release,判定取部署 binary 腿):

用例数 结果 tilingKey
冒烟 12 12/12 PASS 1
支持面闸门 105 105/105 PASS 1
列条纹分支 case00408(fp32,x=(1,1,20000)) 1 1/1 PASS 0

合计 118/118 PASS;bin_compile_s 全部 BINARY_MATCH,越界检查 bin_oob_resultOK,OKmemory_oob_status 全 PASS,两个 tilingKey 分支均已覆盖。

文档更新

无。

类型标签

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

变更摘要

本 PR 针对 arch35(Ascend950)平台的 segsum 算子内核头文件进行改名:将 math/segsum/op_kernel/arch35/ 目录下的头文件 segsum.h 重命名为 segsum_arch35.h,以避免与 A2 平台同名的头文件产生冲突,并同步更新了引用该头文件的源码及文件注释。

主要改动

  • 头文件重命名:将 math/segsum/op_kernel/arch35/segsum.h 重命名为 segsum_arch35.h,通过文件名带 arch35 后缀区分不同架构实现,避免与 A2 平台同名头文件冲突。
  • 更新 include 引用:在 math/segsum/op_kernel/arch35/segsum.cpp 中将 #include "segsum.h" 改为 #include "segsum_arch35.h",保证 arch35 内核入口能正确引用改名后的头文件。
  • 同步文件注释:在 segsum_arch35.h 中将 \file segsum.h 的注释更新为 \file segsum_arch35.h,保持文档描述与实际文件名一致。
likedislike
不准确?
atomgit-bot
atomgit-bot
13 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
13 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
13 天前 评论:

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
math/segsum 汤平川, 宋恺 (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

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

likedislike
CANN-robotCANN-robot成员
13 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,renruhai,rxtfeng,luowen203_gg123,loov1,zhangzijie,jia0liang,songkai111,chenxingyu18,liujie12345678,demoauguste,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为评审人
CANN-robotCANN-robot成员
13 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,renruhai,rxtfeng,luowen203_gg123,loov1,zhangzijie,songkai111,chenxingyu18,liujie12345678,demoauguste,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding设为审查人
Zzl_hw成员
13 天前 修改标题为 “refactor(segsum): arch35 内核头文件更名为 segsum_arch35.h,消除与 A2 同名”,原标题为“fix(segsum): arch35 内核头文件改名 segsum_arch35.h,避免与 A2 同名”
Zzl_hw成员
13 天前 修改了pull request 的描述
zl_hw成员
13 天前 评论:

compile

likedislike
Zzl_hw成员
13 天前 update merge request[project id: 7649531, iid: 5035, commit_id: 0c2786a7d84e16f8a7b3338579ffb98e647ff9de] virtual merging success
CANN-robotCANN-robot成员
13 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
13 天前 评论:

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

任务名称状态日志下载链接
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 ✅ SUCCESS >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>

[2026-08-25 15:41:32]    CI执行结束

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

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

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

[2026-08-25 15:36:18]    CI执行结束

likedislike
CANN-robotCANN-robot成员
13 天前 添加了label:api-check-pass
CANN-robotCANN-robot成员
13 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
13 天前 添加了label:ci-pipeline-passed
TangPC
TangPC成员
13 天前 评论:

/lgtm
/approve

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

/lgtm

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