已合并
fix: 拦截不支持ScalarData的codegen输入(#161) #1209
fix: 拦截不支持ScalarData的codegen输入(#161) #1209
已合并
zqmin创建于 7月6日
zqmin
zqmin成员
7月6日

Pull Request

描述

修复 BitwiseAnd 等声明不支持 scalar input 的 codegen 校验漏拦截 ScalarData 的问题。

本次将 ScalarData 纳入 IsNodeHasScalarInput / IsNodeFirstInputScalar 判定,使不支持 scalar input 的节点在 CheckGraphValidity 阶段提前失败,避免继续生成非法 AscendC 代码并在设备编译时报 ScalarData[0] 语法错误。

变更类型

请选择本次引入的变更类型:

关联的Issue

关联 #161

如何测试

描述测试此变更的步骤和前提条件:

  1. cmake --build build --target aihac_codegen -j 8 通过。
  2. sh build.sh -u --module=autofuse_framework -j 8 本地因 CANN 环境缺失 SkSystemArgs / latest version.info 阻塞,已记录在其他信息中,完整 UT 依赖 CI 流水线验证。

核对清单

其他信息

本地完整 framework UT 未完成,环境报错如下:

  • sk_entry.asc: no type named 'SkSystemArgs' in namespace 'sk'
  • /usr/local/Ascend/ascend-toolkit/latest/share/info/*/version.info does not exist

本 PR 仅修改 scalar input 判定和新增 codegen graph check 单测,不涉及运行时 ABI/API。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zqmin 的贡献)
zqminzqmin成员
7月6日 创建了 pull request,commit 0c4cba30
zqminzqmin成员
7月6日 关联了issue:BitwiseAnd ScalarData 输入未在 codegen 阶段拦截
CANN-robotCANN-robot成员
7月6日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7月6日 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
7月6日 评论:

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


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/graph-autofusion ✅ zhanj, wangxiaotian995, xchu42 (3/2) ✅ zhanj (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了94条消息 查看更多
xchu42
xchu42成员
7月6日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
7月6日 添加了label:lgtm
zhanj
zhanj成员
7月7日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
7月7日 添加了label:approved
CANN-robotCANN-robot成员
7月7日 合入了pull request