已合并
docs: 修复注释与代码不一致(asinh dtype 范围与 cdist UB 注释) #6492
docs: 修复注释与代码不一致(asinh dtype 范围与 cdist UB 注释) #6492
已合并
TangPC创建于 10 天前
TangPC
TangPC成员
10 天前

描述

  • math/asinh/op_host/asinh_def.cpp:注释改为 FP32/FP16/BF16 均已实现。
  • math/cdist/op_host/arch35/cdist_tiling_arch35.cpp:HL fp32 平面注释由 4096 元素/32KB 修正为 8192 元素/每平面 32KB(共 64KB)。

关联的Issue

测试

  • pre-commit(含 OAT):通过
  • Markdown 链接/图片检查:无新增失效项
  • Markdown 代码围栏检查:通过
  • 全部 26 个分支合并验证:无冲突

文档更新

  • math/asinh/op_host/asinh_def.cpp
  • math/cdist/op_host/arch35/cdist_tiling_arch35.cpp

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 TangPC 的贡献)
TangPCTangPC成员
10 天前 创建了 pull request,commit 0df2749a
atomgit-bot
atomgit-bot
10 天前 评论:

变更摘要

本 PR 为纯文档/注释修正,不涉及可执行逻辑变更,主要修复 math/asinh 与 math/cdist 两个算子中注释与代码实际行为不一致的问题。math/asinh/op_host/asinh_def.cpp 的 dtype 支持范围注释由“仅 FP32 完整实现、FP16/BF16 迭代二落地”更新为 FP32、FP16、BF16 均已实现;math/cdist/op_host/arch35/cdist_tiling_arch35.cpp 修正了 DoTiling() 中关于高精度(HL)fp32 平面 UB 预留的规模描述,使元素数量与显存占用说明同实际预留逻辑保持一致。

主要改动

  • asinh dtype 范围注释更新: 将 asinh_def.cpp 头部注释从“迭代一范围:仅 FP32 完整实现;FP16/BF16 在迭代二落地”改为“FP32、FP16、BF16 均已实现”,使 dtype 说明与当前实现状态一致。
  • 保留 DataType 列表声明说明: 注释中继续说明 DataType 列表声明 3 个 dtype 是为了保证 TilingKey 接口稳定,仅去除与迭代阶段相关的过时表述。
  • 修正 HL fp32 平面元素数注释: cdist_tiling_arch35.cpp 中 DoTiling() 的注释由 x1exp + diff 各 4096 元素修正为各 8192 元素,与实际计算平面规模对齐。
  • 补全平面 UB 占用与总量描述: 将原来的“32KB”说明细化为每平面 32KB、两平面共 64KB,并保留其与 fp32 M-split 累加器余量共同预留以避免大 P·R tile 下 UB 溢出的语义。

🧩 本次 Review 已使用自定义 Skill:

likedislike
不准确?
atomgit-bot
atomgit-bot
10 天前 评论:

代码审查

✅ 未发现问题


🧩 本次 Review 已使用自定义 Skill:

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

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
*/*/op_host/*_def.cpp ✅ 王永光, 王林木, 张磊 (3/2) ✅ 王永光 (1/1)
repo-cann/ops-math ✅ 王林木, 张磊 (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

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

likedislike
CANN-robotCANN-robot成员
10 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,tang-lei01,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,llimwang,gubaocheng,andong_hw,wangyongguang,xuejinghui,zhanw_coding,clinglai0517设为评审人
CANN-robotCANN-robot成员
10 天前 将yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,tang-lei01,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,llimwang,gubaocheng,andong_hw,wangyongguang,xuejinghui,zhanw_coding,clinglai0517设为审查人
TangPC
TangPC成员
7 天前 评论:

compile

likedislike
TangPCTangPC成员
7 天前 预合并成功(commit_id: bc18674aef2fd2a1e430821263dcb502c5de29b7)
CANN-robot
CANN-robot成员
7 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
7 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
7 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
7 天前 添加了label:ci-pipeline-passed
chenyifeng成员
7 天前 评论:

您好,PR已满足合入指标,请尽快联系Committer进行合入。如暂时不需要合入,请在PR标题前加[WIP]

likedislike
TangPCTangPC成员
7 天前 修改了pull request 的描述
llimwang成员
7 天前 评论:

/lgtm
/approve

likedislike
zl_hw成员
7 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
7 天前 添加了label:lgtm
wangyongguang成员
7 天前 评论:

/approve

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