已开启
[9.2.0] fix(aicpu): 补充5处成员函数const修饰(G.CNS.03残留) #5903
[9.2.0] fix(aicpu): 补充5处成员函数const修饰(G.CNS.03残留) #5903
已开启
liu-wei创建于 15 天前
liu-wei
liu-wei成员
15 天前

描述

将 PR #5897(5 处 G.CNS.03 成员函数 const 修饰)回移植到 9.2.0 分支,共 8 个文件 +10/-10,声明与定义同步修改:

告警 文件 修复
G.CNS.03 ×1 strided_slice_v2_aicpu.h/.cpp CheckParam 补充 const
G.CNS.03 ×2 trans_data_aicpu.h/.cpp DealData/PaddingTwo 补充 const(保留 9.2.0 声明形参名,仅加 const)
G.CNS.03 ×1 real_div_aicpu.h/.cpp RealDivSameTypeCompute 补充 const
G.CNS.03 ×1 mul_aicpu.h/.cpp MulSameTypeCompute 补充 const

与 master 的差异:trans_data_aicpu.h 的 DealData 声明在 9.2.0 上保留本分支形参名(input_Data/out_put_tensor),仅追加 const;master 上 GetPaddingOneShape 的 const 来自另一提交,不在本次回移植范围。

未包含项

  • trans_data 的 DispatchDealData G.CNS.04 ×3:维持误报结论(同 PR #5856/#5897);
  • strided_slice 的 G.EXP.27 ×4:由 #3267 另行跟进。

验证

  • 二进制级对比(父提交 9c7c1c0f8 vs 本提交 502bd4eee,同编译参数 g++ -O2 -std=c++14):4 个改动 TU 的全部非符号节(共 4598 节,含 trans_data 4 个 _ZN→_ZNK 重命名节配对比对)逐字节相同,符号差异仅为 5 处 const 化函数修饰名——零功能影响;
  • CI 流水线待跑。

关联的Issue

https://gitcode.com/cann/ops-math/issues/3272

类型

🧹 代码清理

likedislike
合并受阻
liu-weiliu-wei成员
15 天前 创建了 pull request,commit 96ebb5f3
liu-weiliu-wei成员
15 天前 关联了issue:[Requirement|需求建议]: 清理 strided_slice AICPU 有符号数位运算告警(G.EXP.27 ×4),[Requirement|需求建议]: 清理 AICPU 算子 G.CNS.03 成员函数 const 告警残留(5 处)
liu-wei
liu-wei成员
15 天前 评论:

compile

likedislike
liu-weiliu-wei成员
15 天前 virtual merging failed, update merge request[project_id: 7649531, iid: 5903, target_commit_sha: 96ebb5f37cd1b40ac6842eb6c9d8b101a77c86a5], message: Conflict detected
liu-weiliu-wei成员
15 天前 virtual merging failed, update merge request[project_id: 7649531, iid: 5903, target_commit_sha: 96ebb5f37cd1b40ac6842eb6c9d8b101a77c86a5], message: Conflict detected
CANN-robot
CANN-robot成员
15 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
15 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
15 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
15 天前 添加了label:ci-pipeline-failed
liu-wei
liu-wei成员
15 天前 评论:

compile

likedislike
liu-weiliu-wei成员
15 天前 virtual merging failed, update merge request[project_id: 7649531, iid: 5903, target_commit_sha: 96ebb5f37cd1b40ac6842eb6c9d8b101a77c86a5], message: Conflict detected
liu-weiliu-wei成员
15 天前 virtual merging failed, update merge request[project_id: 7649531, iid: 5903, target_commit_sha: 96ebb5f37cd1b40ac6842eb6c9d8b101a77c86a5], message: Conflict detected
liu-wei
liu-wei成员
15 天前 评论:

/compile cancel

likedislike
liu-weiliu-wei成员
15 天前 virtual merging failed, update merge request[project_id: 7649531, iid: 5903, target_commit_sha: 96ebb5f37cd1b40ac6842eb6c9d8b101a77c86a5], message: Conflict detected
liu-weiliu-wei成员
15 天前 virtual merging failed, update merge request[project_id: 7649531, iid: 5903, target_commit_sha: 96ebb5f37cd1b40ac6842eb6c9d8b101a77c86a5], message: Conflict detected
CANN-robot
CANN-robot成员
15 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
15 天前 删除了label:ci-pipeline-failed
CANN-robotCANN-robot成员
15 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
15 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
15 天前 删除了label:ci-pipeline-running
此处折叠了7条事件消息 查看更多
CANN-robotCANN-robot成员
15 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
15 天前 评论:

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

⚠️ This PR does not yet meet the following requirements:approve (requires ≥ 1 person(s) per module)

Module Approval Details

module lgtm status approve status
9.2.0 ✅ yue-ma, 王瑞 (2/2) ❌ (0/1)(You can also ask: jiangtao_rts, 陈娇, 田晓亮, gcw_kUomxQ2l, Andy-lb)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
15 天前 将FelixTang7,yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,TianXL,jia0liang,zl_hw,songkai111,jiangtao_rts,gitcode-chenjiao,chenxingyu18,hanxiaolong,pingchuantang,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding,gcw_kUomxQ2l,Andy-lb设为评审人
CANN-robotCANN-robot成员
15 天前 将TianXL,jiangtao_rts,gitcode-chenjiao,hanxiaolong,gcw_kUomxQ2l,Andy-lb设为审查人
liu-wei
liu-wei成员
15 天前 评论:

compile

likedislike
liu-weiliu-wei成员
15 天前 预合并成功(commit_id: 6b18d99fd65ab45bb6de3bc1a979ab6653f3cd83)
CANN-robot
CANN-robot成员
15 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
15 天前 删除了label:ci-pipeline-failed
CANN-robotCANN-robot成员
15 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
15 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
15 天前 添加了label:ci-pipeline-passed
RuiWang_成员
15 天前 评论:

/lgtm
/approve

likedislike
yue-ma成员
15 天前 评论:

/lgtm
/approve

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

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
/approve: 王瑞, yue-ma

likedislike
sunchun成员
15 天前 评论:

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

likedislike