已合并
revert: restore TMULA ISA API #1558
zhywang创建于 8月28日
revert: restore TMULA ISA API #1558
已合并
zhywang创建于 8月28日
zhywang
8月28日

Created-by: zhhywang
Commit-by: zhhywang
Merged-by: cann-robot
Description: 回退 TMULADDDST ISA API 恢复提交,在公共接口、后端、ST、文档和合规配置中统一恢复 TMULA,并完善迁移索引。

描述

40 个文件,新增 294 行、删除 304 行。

  • 范围:更新公共指令接口与 event、CPU/A2A3/A5 后端实现及对应 ST,同步 ISA 文档、MkDocs、索引与 SVG 生成配置,并完善 OAT 数据文件过滤规则。
  • 问题:MR !1538 恢复了 TMULADDDST 名称,与既定的融合乘加指令重命名结果不一致;同时迁移索引未完整说明 TMULADDDST 到 TMULA 的映射,严格 JSON 格式的 manifest 也会被 OAT 误判为缺少版权头。
  • 修复:回退 TMULADDDST 恢复改动,将公开接口、实现符号、文件名、测试用例和文档统一恢复为 TMULA;补齐索引生成器中的双重命名迁移说明,并对 YAML/YML 数据文件关闭版权头扫描。
  • 影响:公开 API 从 TMULADDDST 恢复为 TMULA;兼容性文档保留名称迁移说明,不提供旧名称 wrapper;严格 JSON 格式的 manifest 保持可被现有生成脚本读取。

关联的Issue

https://gitcode.com/cann/pto-isa/merge_requests/1538

测试

  • gitcode_commit.py precheck --repo .
    • 结果:PASS,pre-commit 检查通过,完整 OAT 校验通过 26 个文件。
  • python3 docs/tools/gen_isa_indexes.py --check
    • 结果:PASS,ISA 索引与 manifest 同步。
  • python3 docs/tools/gen_isa_svgs.py --check
    • 结果:PASS,指令 SVG 文件完整。
  • bash -n tests/run_st.sh
    • 结果:PASS。

文档更新

已同步更新 ISA 中英文文档、指令清单、MkDocs、索引生成配置、兼容性说明和指令图。

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhywang 的贡献)
Zzhywang
8月28日 创建了 pull request,commit 0c9a2e55
atomgit-bot
atomgit-bot
8月28日 评论:

变更摘要

本 PR 回退 MR !1538 对 TMULADDDST 名称的恢复改动,将融合乘加指令在公共接口、后端实现、ST 测试与 ISA 文档中统一恢复为 TMULA:公开 API 由 TMULADDDST 还原为 TMULA,不再提供旧名称 wrapper,兼容性文档仅保留 TMULADDDST 更名为 TMULA 的迁移说明。

主要改动

  • 公共接口与枚举恢复:include/pto/common/event.hpp 的 Op 枚举及 PTO_DEFINE_OP_PIPE 映射中删除 TMULADDDST、新增 TMULA;pto_instr.hpp(公共与 costmodel 版本)新增 TMULA 指令函数并移除 TMULADDDST,同时 pto_instr_impl.hpp 的 include 列表同步切换为 TMula.hpp。
  • 后端实现符号统一:CPU 侧 ElementOp::OP_MULADDDST 改为 OP_MULA、ElementTileOp.h 的 BINARY_OP_DEF(MULADDDST) 改为 BINARY_OP_DEF(MULA);A2A3/A5 的 TMulAddDst.hpp 重命名为 TMula.hpp,内部结构 MulAddDstOp、函数 TMulAddDst、TMULADDDST_IMPL 及断言信息统一更名为 MulaOp、TMula、TMULA_IMPL、TMULA。
  • ST 测试与脚本更新:A2A3 测试目录 tmuladddst 重命名为 tmula,测试类 TMULADDDSTTest、启动函数 LaunchTMULADDDST、kernel runTMULADDDST 及用例名全部改为 TMULA 命名;tests/run_st.sh 中 A2A3/A5 的用例参数同步改为 tmula/TMULATest,A5 下新增 tmula 目录链接并删除 tmuladddst 链接。
  • 文档与生成配置同步:docs/isa/manifest.yaml 将指令条目由 TMULADDDST 恢复为 TMULA,mkdocs.yml、gen_pages.py 导航及 gen_isa_svg_core.py 三元表达式配置同步恢复 TMULA;gen_isa_indexes.py 的中英文兼容性说明更新为 TMULADDDST 更名为 TMULA,保留名称迁移说明。
likedislike
不准确?
atomgit-bot
atomgit-bot
8月28日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
zhywang
8月28日 评论:

/compile

likedislike
Zzhywang
8月28日 预合并成功(commit_id: 255d3cfc12030279dce02ae2c1ea967992aececa)
CANN-robotCANN-robot成员
8月28日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
8月28日 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
8月28日 评论:

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)

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
8月28日 将zhangjian_hz11,csjlchen,bluesky901,changxianyu,Zhanghaijian,syfeng,shengxinchen,yinjie27_,zhoubotcam,HuangXingYuan_777,linjiashu,WenboCodes,qukelin,ChanKaLok,kerwinair,echo_stone设为评审人
CANN-robotCANN-robot成员
8月28日 将zhangjian_hz11,csjlchen,bluesky901,changxianyu,Zhanghaijian,syfeng,shengxinchen,yinjie27_,zhoubotcam,HuangXingYuan_777,linjiashu,WenboCodes,qukelin,ChanKaLok,kerwinair,echo_stone设为审查人
CANN-robotCANN-robot成员
8月28日 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
8月28日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
8月28日 添加了label:ci-pipeline-failed
代码行星
代码行星成员
8月28日 评论:

/approve

likedislike
zhywang
8月28日 评论:

/compile

likedislike
Zzhywang
8月28日 预合并成功(commit_id: 073c9e0613d1555266776930dd576b2b567b37aa)
CANN-robotCANN-robot成员
8月28日 删除了label:ci-pipeline-failed
CANN-robotCANN-robot成员
8月28日 添加了label:ci-pipeline-running
zhangjian_hz11成员
8月28日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
8月28日 添加了label:approved
此处折叠了6条事件消息 查看更多
CANN-robotCANN-robot成员
8月28日 删除了label:approved
CANN-robot
CANN-robot成员
8月28日 评论:

Notice

New code changes of the pull request are detected and remove these labels approved. 😳

likedislike
CANN-robotCANN-robot成员
8月28日 添加了label:cann-cla/yes
Zzhywang
8月28日 修改了pull request 的描述
zhywang
8月28日 评论:

/compile

likedislike
Zzhywang
8月28日 预合并成功(commit_id: 9eb5f00842abf088af55b166d4b500b6cdc8bc58)
CANN-robotCANN-robot成员
8月28日 删除了label:ci-pipeline-failed
CANN-robotCANN-robot成员
8月28日 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
8月28日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
8月28日 添加了label:ci-pipeline-passed
zhangjian_hz11成员
8月29日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
8月29日 添加了label:approved
shengxinchen成员
8月29日 评论:

/approve

likedislike
WenboCodes成员
8月29日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
8月29日 添加了label:lgtm
CANN-robotCANN-robot成员
8月29日 合入了pull request