已合并
回退 pr 5080 #5117
zzzzx9创建于 23 天前
回退 pr 5080 #5117
已合并
zzzzx9创建于 23 天前
zzzzx9
zzzzx9成员
23 天前

描述

避免造成MC样例报错的影响。

关联的Issue

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zzzzx9 的贡献)
zzzzx9zzzzx9成员
23 天前 创建了 pull request,commit 527a2705
atomgit-bot
atomgit-bot
23 天前 评论:

变更摘要

本 PR 回退了 PR #5080 中的改动,目的是避免其对 MC 样例造成报错影响。核心变更集中在 impl/adv_api/detail/hccl/cc/src/ops/op_common/executor/executor_v2_base.h,删除了 executor_v2_base 类中两个新增的 ABI 占位虚函数声明,使该头文件恢复到回退前的接口定义状态。

主要改动

  • 删除 CalcCostCoeffAbiPlaceholder 虚函数声明: 移除了 virtual void* CalcCostCoeffAbiPlaceholder(void*, void*, const char*) { return nullptr; },该接口不再作为 executor_v2_base 的成员暴露。
  • 删除 GetAlgNetMetaAbiPlaceholder 虚函数声明: 移除了 virtual void* GetAlgNetMetaAbiPlaceholder(void*) const { return nullptr; },相应占位接口从基类中清理,避免相关 ABI 占位改动引发 MC 样例报错。
likedislike
不准确?
atomgit-bot
atomgit-bot
23 天前 评论:

代码审查

✅ 未发现问题

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

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
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
repo-cann/asc-devkit kong0808, 苏建加 (2/2) kong0808 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了130条消息 查看更多
CANN-robotCANN-robot成员
22 天前 添加了label:approved
suqwe
suqwe成员
22 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
22 天前 添加了label:lgtm
CANN-robotCANN-robot成员
22 天前 添加了label:ci-confirmedci-pipeline-passed;删除了label:ci-pipeline-failed
CANN-robotCANN-robot成员
22 天前 合入了pull request