已合并
[feature]根据断点位置自动下发软硬断点 #130
wangyixian创建于 5月8日
[feature]根据断点位置自动下发软硬断点 #130
已合并
wangyixian创建于 5月8日
wangyixian成员
5月8日

1. 修改描述

  • 修改原因:
  1. 当前simd_vf, simt_vf上只支持硬断点设置,用户需要自己识别并修改断点设置参数,本mr用于自动识别断点位置是否处于simt_vf, simd_vf来
  2. 当前断点ID会由于多个相同的kernel object递增,这里做了删除操作。
  • 修改方案: 编译器提供每个函数的类型信息,simd/simt等,msdebug解析出pc所在的函数类型信息。具体地:在Function类里增加function_class字段,表示函数类型信息;在设置断点的时候,获取pc的SymbolContext里的function类型,识别出是simt/simd函数类型后,修改断点类型。
  • 修改内容:
  1. lldb/include/lldb/Symbol/Function.h 里的Function类里增加function_class字段,每个bit有自己的语义,和编译器对齐
  2. 增加SymbolContext的判断在Breakpoint::AddLocation函数里,这个位置属于比较底层了,函数名/文件名行号/地址,断点解析后都会走到这里
  3. 增加函数将NotifyModuleUpdated能力释放出来,使用新的module替换老的module, 这样当用户没跑算子时,工具识别到的断点和运行后工具产生的断点能够合并。因为后者才是真正的断点

2. 功能验证

  • image.png

  • image.png


3. 代码检视

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

4. 资料修改自检

  • 资料修改: NA

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 wangyixian 的贡献)
ascend-robotascend-robot成员
5月8日 添加了label:stat/needs-squash
ascend-robot
ascend-robot成员
5月8日 评论:

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) 龚思维 (1/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成员
5月8日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
5月8日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
5月8日 添加了label:docs-ci-pipeline-running
此处折叠了96条消息 查看更多
ascend-robotascend-robot成员
5月9日 添加了label:ci-pipeline-passed
gong-siwei成员
5月11日 评论:

/lgtm
/approve

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

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月11日 合入了pull request