Star727
1027
代码
代码
Issues60
Pull Requests204
Actions
项目讨论
Wiki
分析
Star727
1027
已合并
fix: 修正矩阵转置样例类型及文档 #5067
liuyi424创建于 24 天前
fix: 修正矩阵转置样例类型及文档 #5067
已合并
liuyi424创建于 24 天前
从liuyi424/asc-devkitcodex/fix-simd-simt-matrix-transpose合入到cann/asc-devkitmaster
L
liuyi424成员
24 天前

描述

修正 simd_simt_matrix_transpose 样例中的类型和文档一致性问题：

  • 将样例中的裸 int 类型替换为定宽整数类型，并同步修正格式化输出。
  • 保持中英文README代码片段与源码一致，英文代码注释使用对应的英文翻译。
  • 补齐英文README缺失的仿真性能采集内容，修正命令、标题层级和图号。
  • 补充样例实际运行输出，统一中英文README结构。

关联的Issue

无。

测试

  • git diff --check通过。
  • 中英文README标题、列表、表格、图片、链接和代码块结构一致。
  • 中英文README中的5个C++代码片段除注释语言外完全一致，并按源码顺序匹配。
  • 远端Git hooks检查通过。

文档更新

更新了样例的 README.md 和 README_en.md。

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
（感谢 liuyi424 的贡献）
Lliuyi424成员
24 天前 创建了 pull request，commit 7e7a8c4c
atomgit-bot
atomgit-bot
24 天前 评论：

变更摘要

该 PR 主要修复 simd_simt_matrix_transpose 样例中类型与文档的一致性问题。源码层面将裸 int 类型替换为定宽整数类型 uint32_t，并同步修正 main 函数签名与格式化输出；文档层面同步更新 README.md 和 README_en.md，使中英文代码片段与源码一致、补齐英文文档缺失的仿真性能采集内容，并统一标题层级、图号与结构。

主要改动

  • 常量类型统一为定宽整数: 将 TILE_DIM、TILE_PAD、TILE_PAD_STRIDE、TILES_PER_BLOCK 的类型由 int 改为 uint32_t，与尺寸、线程数等无符号语义保持一致。
  • main 函数签名修正: 将 main 的返回类型及 argc 参数类型由 int 改为 int32_t。
  • 格式化输出同步修正: 将错误提示 "Matrix shape must be multiple of TILE_DIM=%d.\n" 中的 %d 改为 %u，以匹配 TILE_DIM 的 uint32_t 类型。
likedislike
atomgit-bot
atomgit-bot
24 天前 评论：

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
24 天前 添加了label：cann-cla/yes
CANN-robot
CANN-robot成员
24 天前 评论：

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
examples ✅ ai_xin, louyuxuan (2/2) ✅ ai_xin (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
24 天前 将zengjuan,zhanghao_0689,suqwe,wulinyu,kong0808,dengjing_aoe,YeZZzzz1,bluesky901,yangbinhua,zhudi22,chenyiyuan,xun_zhuge,XuebinYang,sjtulxh,bianque,houyuyang,Lycheeeee,YANXI_ZHAO,chentianyu19,shi_nanxiang,wuyang_hw,jiangxinyu3,munanhw,ai_xin,ascendhjh,hujiawen_kaven,wangxiaozhi,DragonBornHD84,louyuxuan,changxianyu,hyolee,yukangchen设为评审人
CANN-robotCANN-robot成员
24 天前 将xun_zhuge,bianque,YANXI_ZHAO,munanhw,ai_xin,changxianyu设为审查人
G
gao_dafa成员
24 天前 评论：

liuyi424，感谢您的提交，建议在完成PR充分自验证后，再触发线上构建任务。

若您的PR已具备检视条件，可根据下表的责任田，@对应的committer进行代码检视，我们会在第一时间对您的提交进行处理。

代码责任田：

module committer
adv_api chenyiyuan, wuyang_hw
basic_api chenyiyuan, wuyang_hw
c_api pangfd, haipingmao
cmake xun_zhuge, DragonBornHD84
docs munanhw, ai_xin
examples changxianyu, bianque
simt_api wulinyu, zhanghao_0689
scripts xun_zhuge, kevin365
tools YANXI_ZHAO, XuebinYang, sjtulxh

💡 如您不确定可以@谁，可以直接@我或者ascend-c sig的maintainer，xun_zhuge和ascendhjh，我们会为您分配合适的committer进行代码检视。

再次感谢您参与CANN的社区贡献。

likedislike
Lliuyi424成员
24 天前 update merge request[project id: 8547062, iid: 5067, commit_id: 87fc2c421693e7d6c444bc5b76120c16281c6ab5] virtual merging success
此处折叠了5条事件消息 查看更多
Lliuyi424成员
24 天前 修改了pull request 的描述
L
liuyi424成员
24 天前 评论：

compile

likedislike
Lliuyi424成员
24 天前 update merge request[project id: 8547062, iid: 5067, commit_id: 4d4a7e22fb482216699579b0ac22e0e20c9c51f8] virtual merging success
CANN-robotCANN-robot成员
24 天前 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
24 天前 评论：

流水线任务触发成功
任务链接 [f8227144dcdf475da2b32cb58faac316][流水线指导]

任务名称状态日志下载链接
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck ✅ SUCCESS >>>>>
codecheck_Pr ✅ SUCCESS
pre_comment ✅ SUCCESS >>>>>
StaticCheck_codespell ⚠️ WARNING >>>>>
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_codestyle ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_md ⚠️ WARNING >>>>>

[2026-08-13 15:49:05]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
24 天前 评论：

流水线任务触发成功
任务链接 [c8001365ef4e453ab76f2235d049b236][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_adv_test_one ✅ SUCCESS
UT_Test_adv_test_two ✅ SUCCESS
UT_Test_basic_test_one ✅ SUCCESS
UT_Test_basic_test_two ✅ SUCCESS
UT_Test_basic_test_three ❌ FAILED
UT_Test_basic_test_four ✅ SUCCESS
UT_Test_basic_test_five ✅ SUCCESS
UT_Test_arm ✅ SUCCESS
API_Check ✅ SUCCESS >>>>>
likedislike
CANN-robot
CANN-robot成员
24 天前 评论：

流水线任务触发成功
任务链接 [fa13f4e6fb9840a8b0fe518a70448feb][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
PreSmoke_A900_npupool ❌ FAILED >>>>>
likedislike
W
wulinyu成员
24 天前 评论：

/lgtm

likedislike
Z
zhangshunwei2成员24 天前进行代码检视2
examples/05_simd_simt_hybrid/02_best_practices/simd_simt_matrix_transpose/README.md
已过期
@@ -154,9 +154,10 @@ Case 1通过UB中转把非连续GM写转换为UB内转置访问，Task Duration
154154```cpp
155155uint32_t loop_start = block_idx * TILES_PER_BLOCK;
156156uint32_t loop_step = block_num * TILES_PER_BLOCK;
157+uint32_t loop_end = total_tiles;
Z
zhangshunwei224 天前评论：

为什么一定要这个赋值，不能直接用吗

likedislike
System
系统消息系统
24 天前 评论：

changed this line on b277daf5 view diff detail

CANN-robotCANN-robot成员
24 天前 添加了label：api-check-pass
此处折叠了7条事件消息 查看更多
Lliuyi424成员
24 天前 解决了最后一个问题
L
liuyi424成员
24 天前 评论：

compile

likedislike
Lliuyi424成员
24 天前 update merge request[project id: 8547062, iid: 5067, commit_id: 36a570b8ae62fb35d0becec3f5310028c9614725] virtual merging success
CANN-robotCANN-robot成员
24 天前 删除了label：ci-pipeline-failedapi-check-pass
CANN-robotCANN-robot成员
24 天前 添加了label：ci-pipeline-running
CANN-robot
CANN-robot成员
24 天前 评论：

流水线任务触发成功
任务链接 [4dfa9a6785c54c70abdc1487be7cbf83][流水线指导]

任务名称状态日志下载链接
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck ✅ SUCCESS >>>>>
codecheck_Pr ✅ SUCCESS
pre_comment ✅ SUCCESS >>>>>
StaticCheck_codespell ⚠️ WARNING >>>>>
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_codestyle ✅ SUCCESS >>>>>
codecheck_precommit ✅ SUCCESS >>>>>
StaticCheck_md ⚠️ WARNING >>>>>

[2026-08-13 20:37:59]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
24 天前 评论：

流水线任务触发成功
任务链接 [9e4242a15ea5490abaceff58c67e02a1][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_adv_test_one ✅ SUCCESS
UT_Test_adv_test_two ✅ SUCCESS
UT_Test_basic_test_one ✅ SUCCESS
UT_Test_basic_test_two ✅ SUCCESS
UT_Test_basic_test_three ✅ SUCCESS
UT_Test_basic_test_four ✅ SUCCESS
UT_Test_basic_test_five ✅ SUCCESS
UT_Test_arm ✅ SUCCESS
API_Check ✅ SUCCESS >>>>>
UT_Test_report ✅ SUCCESS >>>>>

[2026-08-13 20:48:20]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
24 天前 评论：

流水线任务触发成功
任务链接 [322dce2097e54bb9ba9415494e5390b2][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>> >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>

[2026-08-13 20:51:22]    CI执行结束

likedislike
CANN-robotCANN-robot成员
24 天前 添加了label：api-check-pass
CANN-robotCANN-robot成员
24 天前 删除了label：api-check-pass
CANN-robotCANN-robot成员
24 天前 删除了label：ci-pipeline-running
CANN-robotCANN-robot成员
24 天前 添加了label：ci-pipeline-passed
A
ai_xin成员
23 天前 评论：

/approve

likedislike
CANN-robotCANN-robot成员
23 天前 添加了label：approved
L
louyuxuan成员
23 天前 评论：

/lgtm

likedislike
CANN-robotCANN-robot成员
23 天前 添加了label：lgtm
CANN-robotCANN-robot成员
23 天前 合入了pull request