已开启
[Requirement|需求建议]: pad_v3_grad算子SIMT kernel变量命名规范化治理 #3
xujiachen8创建于 9月14日
xujiachen8
9月16日 评论：
9月16日 评论：
您好，感谢反馈！
需求分析
pad_v3_grad 的 arch35 SIMT 入口会按 circular、reflect/symmetric 等模式分发到对应 kernel；在 pad_v3_grad_circular_simt.h 和 pad_v3_grad_mirror_simt.h 中，mTD_ 同时承担模式对应的 tiling 数据成员，inIdxCnt 又在 1～5 维计算分支和候选索引辅助函数中重复出现，阅读代码时不容易快速区分填充模式、维度分支和索引用途。
调整方向
上游 PR #5656 已合入 master。该 PR 覆盖上述两个 arch35 SIMT 文件，将成员和局部变量分别改为 mCircularTD_/mMirrorTD_、circularCandidateIdxCnt/mirrorReflectIdxCnt/mirrorSymmetricIdxCnt 以及 circularInIdxCnt1~5/mirrorInIdxCnt1~5，同时整理长表达式和注释排版。改动保持索引计算、边界判断、维度分支和算子接口不变；来源 Issue #3172 已关闭。该副本承接同一处理结果，无需进一步操作。
一、背景信息 （必填）
pad_v3_grad 算子 arch35 平台 SIMT kernel（circular/mirror 填充模式，
op_kernel/arch35/pad_v3_grad_circular_simt.h与pad_v3_grad_mirror_simt.h）中存在泛化命名：成员mTD_与局部变量inIdxCnt在多个模板函数（1~5 维计算分支、CalCandidate/ReflectDimOffset/SymmetricDimOffset 等场景）中重复使用，可读性差，不满足代码度量（cmetrics）整洁要求。需进行变量命名规范化治理。二、价值/作用 （必填）
提升 kernel 代码可读性与可维护性，降低 circular/mirror 模式及不同维度分支间的误读风险；满足代码度量（cmetrics）治理要求。
三、设计方案 （必填）
3.1 使能方式（涉及哪些框架：如Aclnn直调、Pytorch训练等）
不涉及变更。算子以 aclnn exclude 方式接入（CMakeLists 中
ACLNNTYPE aclnn_exclude、DISABLE_IN_OPP TRUE），本次重构不改变算子对外接口与使能方式。3.2 总体设计
3.2.1 算子支持的数据类型
bfloat16、float16、float32、int32、int64（ND 格式，以 ascend950/ascend350 平台算子信息配置为准），本次不改变。
3.2.2 host侧设计
无改动。
3.2.3 kernel侧设计
pad_v3_grad_circular_simt.h：mTD_→mCircularTD_；CalCandidate形参 →circularCandidateIdxCnt；SimtComputeCircularOne~Five局部变量 →circularInIdxCnt1~5pad_v3_grad_mirror_simt.h：mTD_→mMirrorTD_；ReflectDimOffset形参 →mirrorReflectIdxCnt；SymmetricDimOffset形参 →mirrorSymmetricIdxCnt；SimtComputeMirrorOne~Five局部变量 →mirrorInIdxCnt1~53.3 支持硬件
算子 COMPUTE_UNIT 为 ascend950、mc62、ascend350（CMakeLists 中 tiling/kernel 均映射 arch35 目录），本次重构作用于 arch35 kernel 源码，对上述平台统一生效。
3.4 算子约束限制
不涉及变更。
💡 备注（选填）
关联 PR：https://gitcode.com/cann/ops-math/pull/5656