已合并
[2.10.0][refactor]CANN ACL headers decouple #36214
Dring创建于 5月20日
[2.10.0][refactor]CANN ACL headers decouple #36214
已合并
D
Dring
5月20日
5月20日
# 【合入来源】
> **如有社区issue，请关联issue链接**\
https://gitcode.com/Ascend/pytorch/issues/2042
> **请勿携带内部流程信息（需求链接、问题单、内部issue等）**
- [✓ ] 需求
- [ ] 问题单
- [ ] issue/工单
- [✓ ] 重构优化
- [ ] 资料更新
# 【修改方案】
> 请描述修改内容的具体实现，涉及哪些组件之间进行交互，可以用1、2、3、...进行罗列\
> 如果是需求或者重构类的PR，需要补充详细设计文档（说明上下游组件关系、时序图、类图、DFX能力等内容）
cann已开源，PTA在编译过程中动态获取CANN头文件，避免本地保存的头文件与CANN开源版本存在差异导致上层应用编译异常。
核心逻辑：
2.10以下分支保留开源头文件中没有的头文件，2.10以上分支原头文件全部删除，拉取的开源头文件进行复制覆盖
2.10以上分支检查 third_party/acl/inc/acl/ 是否已有内容，有则跳过（幂等）
按优先级复制头文件到 third_party/acl/inc/acl/（先 runtime，后 ge 覆盖，再super_kernel.h）
删除临时目录
# 【资料变更】
> 请确认是否涉及资料变更。如涉及，需要在PR中体现，并简要说明修改内容。如不涉及，需填写“不涉及”
不涉及
# 【接口变更】
> 请确认是否涉及跨代码仓或者客户面可见的接口变更。如涉及，需要详细说明接口以及对应的变更内容，同时需要在资料中体现。如不涉及，需填写“不涉及”
不涉及
# 【功能验证】
> 说明测试场景，测试方法。如果本次测试方式与常规单元测试不同，请详细说明您的测试步骤\
> 新增/变更内容是否已新增/适配UT测试用例看护，并补充测试自验证截图
编包成功，没有本地文件找不到的头文件符号
安装成功
初始化无报错：
# 【CheckList】
> PR提交人对以下CheckList自检项进行全量自检，自检通过或不涉及，均修改 [ ] 为 [x]
- [✓ ] 代码注释完备，正确记录错误日志
- [✕ ] 代码实现进行了返回值、空指针等校验
- [✓ ] PR标题正确使用类型标签，如：feat、fix、refactor、docs、test等
- [✓ ] PR持续集成流水线（CI）执行通过，代码检查无异常
Pull Request已成功合入, 合并人@ascend-robot
（感谢 Dring 的贡献）5月20日 添加了label：stat/needs-squash
ascend-robot
5月20日 评论：
5月20日 评论：
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/pytorch | ✅ 李伟, huangyunlong2022 (2/2) | ✅ 李伟 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
Dring, thanks for your pull request. All authors of the commits have signed the CLA. 👍
5月20日 添加了label：ascend-cla/yes
ascend-robot
5月20日 评论：
5月20日 评论：
当前仓库存在以下 保护分支 ：
| Protected Branch | Version | Release |
|---|---|---|
| master | ||
| v2.11.0 | ||
| v2.10.0 | ||
| v2.7.1 | ||
| v2.9.0 | ||
| v2.12.0 |
评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支（创建同步 PR）：
a) 如果当前 PR 是 Open 状态，同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged，将立即执行同步操作
注意：
- /sync 命令可以指定同步到多个分支，仅最后一个 /sync 命令生效
- 如果创建的同步 PR 不正确，可通过向同步 PR 的源分支提交轻量级 PR 完善，或使用 /close 命令关闭
此处折叠了48条消息 查看更多
huangyunlong
5月21日 评论：
5月21日 评论：
/lgtm
5月21日 添加了label：approvedlgtm
ascend-robot
5月21日 评论：
5月21日 评论：
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: 李伟.
Reviewers who wrote a comment of /lgtm are: 李伟, huangyunlong2022.
5月21日 合入了pull request