已合并
[feature] 统一寄存器打印顺序 #124
wangyixian创建于 4月29日
[feature] 统一寄存器打印顺序 #124
已合并
wangyixian创建于 4月29日
wangyixian成员
4月29日

1. 修改描述

  • 修改原因:
  1. 之前的寄存器打印排序算法(先按长度再按字典序)不适用950上的寄存器,比如 S10和S0会被不连续。
  2. coredump 如果没有error寄存器,且是aiv算子,则打印所有拿到的simd/simt寄存器
  3. 修复cube算子生成的core文件core_id展示不对的问题
  4. 修复ctrl-c下,bt命令不能用的问题
  • 修改方案: 按照寄存器地址的顺序打印,这里我们不需要做排序,因为寄存器的定义顺序本身在代码 里就是按照地址顺序来排布的, 并调整PC寄存器为第一个
  • 修改内容:
  1. 删除DumpRegisterSetWithSort,直接使用原生的DumpRegisterSet
  2. 增加UNKNOWN_INTERRUPT_TYPE 类型,用于aiv 算子生成的core文件,没有error 寄存器时,展示所有aiv的寄存器列表,不限制simd/simt
  • 涉及代码双合(贴上另一个PR链接):

2. 功能验证

  • 功能自验截图(请确保不体现个人信息)
    image.png
  • 冒烟是否通过
    image.png

3. 代码检视

  • 要求:
    • 合入功能代码大于 200 行,需要sig会议申报代码检视议题,并在PR中标注会议。
    • committer评估是否需要在sig会议进行代码检视。
    • 参与检视的committer人员名单与检视时间。
    • 大于 1000 行代码原则上不允许合入,需进行备案。
  • 是否经过代码检视
  • 是否具备UT测试用例看护 NA
  • 是否需要在sig会议中进行代码检视 NA
  • 检视committer人员名单与检视时间:

4. 资料修改自检

  • 资料修改: NA

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 wangyixian 的贡献)
ascend-robot
ascend-robot成员
4月29日 评论:

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-Ascend/msdebug 门飞飞, 龚思维 (2/2) 龚思维, 门飞飞 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
4月29日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
4月29日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
4月29日 添加了label:docs-ci-pipeline-running
wangyixian成员
4月29日 评论:

compile

likedislike
此处折叠了122条消息 查看更多
ascend-robotascend-robot成员
5月6日 添加了label:approved
went_code
went_code成员
5月7日 评论:

/lgtm
/approve

likedislike
ascend-robotascend-robot成员
5月7日 添加了label:lgtm
ascend-robot
ascend-robot成员
5月7日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: 门飞飞, 龚思维.
Reviewers who wrote a comment of /lgtm are: 门飞飞, 龚思维.

likedislike
ascend-robotascend-robot成员
5月7日 合入了pull request