已合并
fix: 修复代码检查告警 #4637
duhua创建于 14 天前
fix: 修复代码检查告警 #4637
已合并
duhua创建于 14 天前
duhua
duhua成员
14 天前

Pull Request

描述

修复自定义算子注册、运行时转换及 Host CPU 执行上下文相关代码检查告警:

  1. 统一 OpBackendCustomOpCapability 按值参数的声明与定义。
  2. 修复运行时转换中的 const_cast、旧式 Node/Anchor 接口调用。
  3. 使用 AttrUtils::GetStr 指针重载,减少不必要的字符串拷贝。
  4. 补齐 HostCpuOpExecutionContext 头文件的直接依赖。
  5. 保留动态库符号加载所需的函数指针转换,以及 constant_folding_pass.cc:289 的运行时 ABI 转换,并增加屏蔽说明。

变更类型

关联的Issue

如何测试

  • 执行完整 UT/ST。

核对清单

其他信息

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 duhua 的贡献)
duhuaduhua成员
14 天前 添加了label:enhancement
duhuaduhua成员
14 天前 创建了 pull request,commit 245d0bff
CANN-robotCANN-robot成员
14 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
14 天前 评论:

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
inc/graph_metadef/external 王涛, kobemini (2/2) 王涛 (1/1)
repo-cann/ge 王涛, kobemini (2/2) 王涛 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
14 天前 评论:

CLA Signature Pass

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

likedislike
此处折叠了77条消息 查看更多
CANN-robotCANN-robot成员
12 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
12 天前 添加了label:ci-pipeline-passed
王涛成员
12 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
12 天前 添加了label:lgtmapproved
CANN-robotCANN-robot成员
12 天前 合入了pull request