Merged
fix: restrict sigmoid cross entropy grad tiling to ascend950 #10074
zhouxuan78Add on Sep 8
fix: restrict sigmoid cross entropy grad tiling to ascend950 #10074
Merged
Merged fromDeleted :9.2.0-beta.2intocann/ops-nn9.2.0-beta.2
Pull Request has been successfully merged by CANN-robot.
（Thank you for zhouxuan78's contribution.）Sep 8 Created pull request, commit f39a6b52
Zzhouxuan78
Sep 8 Associated issue: [Bug-Report|缺陷反馈]:sigmoid cross entropy grad的A2实现会走到950的tiling 导致产品线用例失败
Sep 8 Associated issue: [Bug-Report|缺陷反馈]:sigmoid cross entropy grad的A2实现会走到950的tiling 导致产品线用例失败
atomgit-bot
Sep 8 Comments：
Sep 8 Comments：
变更摘要
本PR为Bug修复：此前tiling使用错误，导致A2架构的kernel错误地走入了ascend950的tiling分支，进而造成A2上sigmoid cross entropy grad算子出现精度错误。本次改动将950的tiling全部放入arch35条件中做隔离，从而把该算子的tiling限制为仅适用于ascend950。改动规模为1个文件、新增2行、删除2行。
主要改动
- 限制tiling适用范围：将
sigmoid cross entropy grad的tiling限定为仅用于ascend950，避免其它架构（如A2）的kernel误入950的tiling分支。 - 隔离950 tiling到arch35：将950相关的tiling逻辑全部放入
arch35条件中做架构隔离，修复A2上该算子因tiling使用错误导致的精度问题。
Incorrect?
atomgit-bot
Sep 8 Comments：
Sep 8 Comments：
Sep 8 Modify description of pull request
Sep 8 Added label: cann-cla/yes
CANN-robot
Sep 8 Comments：
Sep 8 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 |
|---|---|---|
| loss | ✅ 王星, 陈琦 (2/2) | ✅ 王星 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
zhouxuan78, thanks for your pull request. All authors of the commits have signed the CLA. 👍
zhouxuan78
Sep 8 Comments：
Sep 8 Comments：
/compile
CANN-robotSep 8 Set zengjuan,chenfeng61,chaotang233,crystalhu,yu-xinjie62,yangyang016,fanqirui,su-yueming,chenqi317,wang-xing001,liubo75,tangweiwei2,pingchuantang,qianzehong as approver
Sep 8 Set zengjuan,chenfeng61,chaotang233,crystalhu,yu-xinjie62,yangyang016,fanqirui,su-yueming,chenqi317,wang-xing001,liubo75,tangweiwei2,pingchuantang,qianzehong as approver
Sep 8 Pre merge succeeded (commit_id: b7a5dd72bcc661c998b89edd7326841e22bf7b19)
CANN-robotSep 8 Set chenfeng61,yu-xinjie62,su-yueming,wang-xing001,pingchuantang,qianzehong as approver
Sep 8 Set chenfeng61,yu-xinjie62,su-yueming,wang-xing001,pingchuantang,qianzehong as approver
CANN-robot
Sep 8 Comments：
Sep 8 Comments：
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |
Sep 8 Added label: ci-pipeline-running
Sep 8 Removed label: ci-pipeline-running
Sep 8 Added label: ci-pipeline-passed
yuning_chen
Sep 8 Comments：
Sep 8 Comments：
您好，pr合入已准备就绪，请尽快联系committer进行检视，谢谢！
lileizheng
Sep 8 Comments：
Sep 8 Comments：
/reject
conv2d_ci
Sep 8 Comments：
Sep 8 Comments：
| Time | Name | Result | Changeid | TimeStamp |
|---|---|---|---|---|
| 2026-09-08 19:57:57 | Start_Pipeline | MR_468 |
wang-xing001
Sep 9 Comments：
Sep 9 Comments：
/lgtm
/approve
Sep 9 Added label: approved
Sep 9 Added label: lgtm
Sep 9 Closed associated issue
Sep 9 Merged pull request
描述
关联的Issue
tiling使用错误，A2的kernel走入950的tiling，导致A2算子sigmoid cross entropy grad的精度出现错误,将950tiling全部放入arch35中，做隔离
测试
https://gitcode.com/cann/ops-nn/issues/5683
文档更新
类型标签
AI/Agent生成声明