已合并
revert: restore TMULADDDST ISA API #1538
zhywang创建于 17 小时前
revert: restore TMULADDDST ISA API #1538
已合并
zhywang创建于 17 小时前
zhywang
17 小时前

Created-by: zhhywang
Commit-by: zhhywang
Merged-by: cann-robot
Description: 回退 TMULADDDST 到 TMULA 的重命名,恢复公共接口、后端实现、文档和测试,同时保留 TMADD 相关修改。

描述

40 files changed, 325 insertions(+), 326 deletions(-)

  • 范围:更新公共指令与事件接口、CPU/A2A3/A5 后端、ISA 文档与生成器、Cost Model facade 以及 CPU/A2A3/A5 ST 用例。
  • 问题:TMULADDDST 到 TMULA 的重命名需要单独回退,同时不能影响 TFUSEDMULADD 到 TMADD 的现有修改。
  • 修复:移除 TMULA 接口及相关路径,恢复 TMULADDDST 的实现、文档、索引、测试目录和测试入口;保留 TMADD 接口及其配套内容。
  • 影响:src0 * src1 + dst 语义的公开 ISA/API 名称恢复为 TMULADDDST;src0 * dst + src1 语义继续使用 TMADD。

关联的Issue

测试

  • python3 docs/tools/gen_isa_indexes.py --check
    • 结果:PASS
  • python3 docs/tools/gen_isa_svgs.py --check
    • 结果:PASS
  • python3 docs/tools/gen_virtual_manual_matrix.py --check
    • 结果:PASS
  • bash -n tests/run_st.sh
    • 结果:PASS
  • gitcode_commit.py precheck --repo .
    • 结果:PASS,所有适用的 pre-commit 检查通过。

文档更新

已恢复 TMULADDDST 的中英文 ISA 文档、索引、导航、兼容性说明和指令图。

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhywang 的贡献)
Zzhywang
17 小时前 创建了 pull request,commit c223d20d
atomgit-bot
atomgit-bot
17 小时前 评论:

变更摘要

该 PR 单独回退了 TMULADDDSTTMULA 的接口重命名,将公开 ISA/API 名称恢复为 TMULADDDST(语义 src0 * src1 + dst),涉及公共指令/事件接口、CPU/A2A3/A5 后端、ISA 文档与生成器、Cost Model facade 以及 CPU/A2A3/A5 的 ST 用例;同时保留 TFUSEDMULADDTMADD 的既有修改(语义 src0 * dst + src1)。

主要改动

  • 恢复公共指令接口:在 include/pto/common/event.hpp 中删除 Op::TMULA 并恢复 Op::TMULADDDST 及对应 PTO_DEFINE_OP_PIPE;在 include/pto/common/pto_instr.hppinclude/pto/costmodel/pto_instr.hpp 中移除 TMULA 模板函数、恢复 TMULADDDST 模板函数;include/pto/common/pto_instr_impl.hpp 中将 TMula.hpp 头文件引用替换回 TMulAddDst.hpp
  • 同步后端实现命名:CPU 端 ElementOp::OP_MULA 改回 OP_MULADDDST(含 ElementOpCal 特化),ElementTileOp.hBINARY_OP_DEF(MULA) 改回 BINARY_OP_DEF(MULADDDST);A2A3/A5 后端的 include/pto/npu/{a2a3,a5}/TMulAddDst.hpp 中将 MulaOpTMulaTMulaCheckTMULA_IMPL 分别恢复为 MulAddDstOpTMulAddDstTMulAddDstCheckTMULADDDST_IMPL
  • 回退 ISA 文档与生成器docs/isa/manifest.yaml 删除 TMULA 指令条目并恢复 TMULADDDSTdocs/mkdocs/gen_pages.pymkdocs.yml 导航中 TMULA.md 替换为 TMULADDDST.mddocs/tools/gen_isa_indexes.py 兼容性说明仅保留 TFUSEDMULADDTMADD 的重命名提示;docs/tools/gen_isa_svg_core.pyTERNARY_ELEMENTWISE_SPECTMULA 改回 TMULADDDST
  • 调整 ST 测试用例与入口:CPU/A2A3 的 tmula 测试目录改名为 tmuladddst,测试类 TMULATest、启动/内核函数 LaunchTMULA/runTMULA 及测试用例名同步恢复为 TMULADDDST 系;A5 侧删除 tmula 符号链接并新增指向 tmuladddst 的链接;tests/run_st.sh 中 A2A3/A5 的 -t tmula -g TMULATest... 运行项替换为 -t tmuladddst -g TMULADDDSTTest...,并移除全量用例列表中的 tmula 条目。
likedislike
atomgit-bot
atomgit-bot
17 小时前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
17 小时前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
17 小时前 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
17 小时前 评论:

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
docs shengxinchen, zhangjian_hz11 (2/2) shengxinchen, zhangjian_hz11 (2/1)
repo-cann/pto-isa zhangjian_hz11, 孙文博 (2/2) zhangjian_hz11, 孙文博 (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
17 小时前 将zhangjian_hz11,csjlchen,bluesky901,changxianyu,Zhanghaijian,syfeng,shengxinchen,yinjie27_,zhoubotcam,HuangXingYuan_777,linjiashu,WenboCodes,qukelin,ChanKaLok,kerwinair,echo_stone设为评审人
CANN-robotCANN-robot成员
17 小时前 将zhangjian_hz11,csjlchen,bluesky901,changxianyu,Zhanghaijian,syfeng,shengxinchen,yinjie27_,zhoubotcam,HuangXingYuan_777,linjiashu,WenboCodes,qukelin,ChanKaLok,kerwinair,echo_stone设为审查人
zhywang
17 小时前 评论:

/compile

likedislike
Zzhywang
17 小时前 update merge request[project id: 8827531, iid: 1538, commit_id: 490e13fbe30424b7f075ace4d779e48e77e3e7ee] virtual merging success
CANN-robotCANN-robot成员
17 小时前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
17 小时前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
17 小时前 添加了label:ci-pipeline-passed
shengxinchen成员
17 小时前 评论:

/approve

likedislike
zhangjian_hz11成员
15 小时前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
15 小时前 添加了label:approved
WenboCodes成员
14 小时前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
14 小时前 添加了label:lgtm
CANN-robotCANN-robot成员
14 小时前 合入了pull request