Merged
refactor(segsum/exp_segsum_grad): 重命名op_kernel头文件区分arch22/arch35 #5615
songkai111Add on Sep 12
refactor(segsum/exp_segsum_grad): 重命名op_kernel头文件区分arch22/arch35 #5615
Merged
Merged fromsongkai111/ops-math_6421masterintocann/ops-mathmaster
Pull Request has been successfully merged by CANN-robot.
（Thank you for songkai111's contribution.）Sep 12 Created pull request, commit fcc3afa3
Sep 12 Added label: cann-cla/yes
CANN-robot
Sep 12 Comments：
Sep 12 Comments：
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 | ✅ 汤平川, zhanw_coding (2/2) | ✅ 汤平川 (1/1) |
| repo-cann/ops-math | ✅ 汤平川, zhanw_coding (2/2) | ✅ 汤平川 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
songkai111, thanks for your pull request. All authors of the commits have signed the CLA. 👍
CANN-robotSep 12 Set FelixTang7,yue-ma,zhou-qilong,wangrui_,renruhai,rxtfeng,luowen203_gg123,loov1,zhangzijie,jia0liang,zl_hw,chenxingyu18,liujie12345678,demoauguste,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding as approver
Sep 12 Set FelixTang7,yue-ma,zhou-qilong,wangrui_,renruhai,rxtfeng,luowen203_gg123,loov1,zhangzijie,jia0liang,zl_hw,chenxingyu18,liujie12345678,demoauguste,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding as approver
CANN-robotSep 12 Set FelixTang7,yue-ma,zhou-qilong,wangrui_,renruhai,rxtfeng,luowen203_gg123,loov1,zhangzijie,jia0liang,zl_hw,chenxingyu18,liujie12345678,demoauguste,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding as approver
Sep 12 Set FelixTang7,yue-ma,zhou-qilong,wangrui_,renruhai,rxtfeng,luowen203_gg123,loov1,zhangzijie,jia0liang,zl_hw,chenxingyu18,liujie12345678,demoauguste,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding as approver
songkai111
Sep 12 Comments：
Sep 12 Comments：
compile
Sep 12 Pre merge succeeded (commit_id: 3f5e751f6502228ea1327a983a6770ee9e436eea)
CANN-robot
Sep 12 Comments：
Sep 12 Comments：
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |
Sep 12 Added label: ci-pipeline-running
6 event messages are collapsed here。 See more
Sep 12 Added label: cann-cla/yes
songkai111
Sep 12 Comments：
Sep 12 Comments：
compile
Sep 12 Pre merge succeeded (commit_id: f70ab1dbfb6c671abccd92d5b99603b9a41a7e48)
CANN-robot
Sep 12 Comments：
Sep 12 Comments：
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |
Sep 12 Removed label: ci-pipeline-failed
Sep 12 Added label: ci-pipeline-running
Ssongkai111
Sep 12 Change title to "refactor(segsum/exp_segsum_grad): 重命名op_kernel头文件区分arch22/arch35", originally titled "refactor(segsum/exp_segsum_grad): 重命名op_kernel头文件区分arch22与arch35".
Sep 12 Change title to "refactor(segsum/exp_segsum_grad): 重命名op_kernel头文件区分arch22/arch35", originally titled "refactor(segsum/exp_segsum_grad): 重命名op_kernel头文件区分arch22与arch35".
Sep 12 Modify description of pull request
Sep 12 Removed label: ci-pipeline-running
Sep 12 Added label: ci-pipeline-passed
TangPC
Sep 14 Comments：
Sep 14 Comments：
/lgtm
/approve
Sep 14 Added label: approved
zhanw_coding
Sep 14 Comments：
Sep 14 Comments：
/lgtm
Sep 14 Added label: lgtm
Sep 14 Closed associated issue
Sep 14 Merged pull request
描述
将
segsum、exp_segsum_grad两个算子的op_kernel头文件按架构重命名，明确区分 arch22 与 arch35，避免不同架构同名头文件在编译期产生歧义；并同步更新引用该头文件的 kernel 入口。具体改动：
exp_segsum_grad/op_kernel/exp_segsum_grad.h→exp_segsum_grad_arch22.h，同步更新exp_segsum_grad.cpp的 include。exp_segsum_grad/op_kernel/arch35/exp_segsum_grad.h→exp_segsum_grad_arch35.h，同步更新arch35/exp_segsum_grad.cpp的 include。segsum/op_kernel/segsum.h→segsum_arch22.h，同步更新segsum.cpp的 include。SEGSUM→SEGSUM_ARCH22_H、EXP_SEGSUM_GRAD_H→EXP_SEGSUM_GRAD_ARCH22_H）及文件头注释。关联的Issue
测试
本改动为头文件重命名与 include 调整，通过编译验证两个算子的 kernel 可正常构建。
文档更新
无
类型标签