已关闭
[Bug-Report|缺陷反馈]: norm/add_rms_norm/op_kernel/add_rms_norm.h 30行 不符合“TilingData 若转换指针使用时必须添加__restrict__  和const修饰符”的编码风格,建议修改 #146
duanhaomiao创建于  2025年11月17日关闭于  2025年11月24日
duanhaomiao
duanhaomiao
2025年11月17日 创建

Thanks for sending an issue! Please fill in the following template to help quickly solve your problem.

Describe the current behavior / 问题描述 (Mandatory / 必填)

代码检视过程中,发现算子源码中存在代码可能存在性能问题。
pooling/adaptive_avg_pool3d/op_kernel/adaptive_avg_pool3d_split_c.h 30行
GM_ADDR x1, GM_ADDR x2, GM_ADDR gamma, GM_ADDR y, GM_ADDR rstd, GM_ADDR x, const AddRMSNormTilingData* __restrict__ tiling)

norm/add_rms_norm/op_kernel/add_rms_norm_merge_n.h 30行
GM_ADDR x1, GM_ADDR x2, GM_ADDR gamma, GM_ADDR y, GM_ADDR rstd, GM_ADDR x, const AddRMSNormTilingData* tiling)

规则描述:
要求TilingData必须不能修改,同时为了配合CCEC编译便于常量折叠和静态编译,必须添加__restrict__ 和 const 修饰符。

Environment / 环境信息 (Mandatory / 必填)

https://gitcode.com/cann/ops-nn/blob/master/norm/add_rms_norm/op_kernel/add_rms_norm.h
https://gitcode.com/cann/ops-nn/blob/master/norm/add_rms_norm/op_kernel/add_rms_norm_merge_n.h

Steps to reproduce the issue / 重现步骤 (Mandatory / 必填)

代码检视即可

Describe the expected behavior / 预期结果 (Mandatory / 必填)

TilingData变量添加__restrict__ 和 const 修饰符。

无

Special notes for this issue/备注 (Optional / 选填)

likedislike
胡一航成员
2025年11月17日 评论:

感谢反馈,正在安排人修改

likedislike
duanhaomiaoduanhaomiao
2025年11月17日 修改标题为 “[Bug-Report|缺陷反馈]: norm/add_rms_norm/op_kernel/add_rms_norm.h 30行 不符合“TilingData 若转换指针使用时必须添加__restrict__  和const修饰符”的编码风格,建议修改”,原标题为“[Bug-Report|缺陷反馈]: pooling/adaptive_avg_pool3d/op_kernel/adaptive_avg_pool3d_split_c.h 30行 34行 不符合“TilingData 若转换指针使用时必须添加__restrict__  和const修饰符”的编码风格,建议修改”
duanhaomiaoduanhaomiao
2025年11月17日 修改了issue 的描述
Lliujie12345678成员
2025年11月18日 将 liujie12345678 设为负责人
duanhaomiaoduanhaomiao
2025年11月18日 issue类型由 任务 改变为 缺陷
CANN-robotCANN-robot成员
2025年11月24日 关闭了 issue
hqhuangaehqhuangae成员
2025年11月28日 添加了label:Accepted
CANN-robotCANN-robot成员
3月20日 添加了label:resolved