已合并
【fix】: 修复Max/Min两个输入都是scalar的bug #1870
【fix】: 修复Max/Min两个输入都是scalar的bug #1870
已合并
liyuewei创建于 8 天前
liyuewei
liyuewei成员
8 天前

Pull Request

描述

1、输入是scalar的场景不再生成brc inline模板。
2、处理两个输入都是scalar的Max/Min api,放到AscendC命名空间,防止和utils.h里的api冲突。

变更类型

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

关联的Issue

如何测试

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

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 liyuewei 的贡献)
liyueweiliyuewei成员
8 天前 添加了label:enhancement
liyueweiliyuewei成员
8 天前 创建了 pull request,commit 06a108d2
atomgit-bot
atomgit-bot
8 天前 评论:

变更摘要

本 PR 为 Bug 修复,核心是对 AscendC 标量 Maximum/Minimum 相关 API 的命名与调用进行统一调整:将标量头文件 scalar_maximum.h / scalar_minimum.h 中的 Maximums / Minimums 重命名为 MaximumExtends / MinimumExtends 并新增带 tensor 的重载,同时将 v1/v2 代码生成器中最小/最大二元运算的 GetApiName() 返回的 AscendC::Min / AscendC::Max 改为新的包装函数 MinimumExtend / MaximumExtend,使代码生成调用与新增 API 保持一致。

主要改动

  • 标量 API 重命名与重载扩展: scalar_maximum.h / scalar_minimum.h 中将标量版 Maximums / Minimums 更名为 MaximumExtends / MinimumExtends,并新增两个重载:tensor 与标量参与、内部调用 AscendC::Maxs / AscendC::MinsMaximumExtends / MinimumExtends,以及 tensor 与 tensor 参与、内部调用 AscendC::Max / AscendC::MinMaximumExtend / MinimumExtend

  • v1 代码生成器调整: v1_ascir_codegen_impl.hMinimumAscIrCodegenImpl / MaximumAscIrCodegenImplGetApiName() 返回值由 AscendC::Min / AscendC::Max 改为 MinimumExtend / MaximumExtend,仍加载 scalar_minimum.h / scalar_maximum.h 头文件。

  • v2 代码生成器调整: v2_ascir_codegen_impl.hMinimumAscIrCodegenImplV2 / MaximumAscIrCodegenImplV2GetApiName() 同步改为 MinimumExtend / MaximumExtend,与 v1 行为保持一致。

likedislike
atomgit-bot
atomgit-bot
8 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:stat/needs-squash
此处折叠了69条消息 查看更多
xchu42
xchu42成员
5 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
5 天前 添加了label:lgtm
张德鹏成员
5 天前 评论:

/approve

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