已合并
[refactor] 升级cann-cmake至master-044，迁移device构建与签名至cann-cmake公共函数 #4176
zhaoyingdong创建于 29 天前
[refactor] 升级cann-cmake至master-044，迁移device构建与签名至cann-cmake公共函数 #4176
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 zhaoyingdong 的贡献）29 天前 创建了 pull request，commit 289e8124
atomgit-bot
29 天前 评论：
29 天前 评论：
变更摘要
本次 PR 主要将 cann-cmake 依赖从 master-042 升级至 master-044，并将原先在本仓库中自行维护的 device 构建配置、交叉编译工具链以及签名/版本参数迁移至 cann-cmake 公共函数统一管理，从而精简本地构建脚本与 CMake 配置。
主要改动
- 升级
cann-cmake版本：在cmake/fetch_cann_cmake.cmake中将CANN_CMAKE_TAG从master-042更新为master-044，同步更新URL_HASH校验值，以匹配新版本分发包。 - 移除本地交叉编译工具链文件：删除
cmake/aarch64-hcc-toolchain.cmake（共 60 行），该文件原先定义 aarch64 HCC 工具链的编译器、链接器、归档器等路径及归档规则，现已迁移至 cann-cmake 公共模块。 - 移除 device 构建/安装路径变量：在
cmake/config.cmake中删除HCOMM_DEVICE_BUILD_PATH和HCOMM_DEVICE_INSTALL_PATH两个 CMake 变量，device 侧构建与安装目录现由 cann-cmake 统一管理。 - 移除
build.sh中的版本与签名自定义参数：删除VERSION_INFO变量、--version命令行选项及其对应的-DVERSION_INFOCMake 传参逻辑，版本与签名能力已迁移至 cann-cmake 公共函数，不再由本地构建脚本直接控制。
atomgit-bot
29 天前 评论：
29 天前 评论：
代码审查
审查总结
逐文件审查结果：
| 文件 | 审查结果 |
|---|---|
| CMakeLists.txt | ✅ 无问题。移除了 ExternalProject_Add 块和未使用的 HCC_C_AR 变量，替换为 add_cann_device_project(hcomm)。HCC_C_COMPILER/HCC_CXX_COMPILER 仍被 ENABLE_BUILD_AARCH 分支使用，删除 HCC_C_AR 正确。 |
| build.sh | ⚠️ 1 个 P3 发现。移除了 VERSION_INFO 默认值、--version 选项解析及其在 CUSTOM_OPTION 中的传递。这是一项 CLI 合约破坏性变更。 |
| cmake/config.cmake | ✅ 无问题。删除了 HCOMM_DEVICE_BUILD_PATH 和 HCOMM_DEVICE_INSTALL_PATH，全仓已无引用。 |
| cmake/device/CMakeLists.txt | ✅ 无问题。为三个 add_cann_sign_file 调用新增了 VERSION "${CANN_VERSION_hcomm_VERSION}" 参数。该变量由 version.cmake 中的 set_cann_package(hcomm VERSION "9.1.0") 生成。 |
| cmake/fetch_cann_cmake.cmake | ✅ 无问题。cann-cmake 标签由 master-042 升级至 master-044，SHA256 哈希值已相应更新。 |
| cmake/aarch64-hcc-toolchain.cmake | ✅ 无问题。已删除文件，全仓无剩余引用。 |
按严重程度统计：
- P0：0
- P1：0
- P2：0
- P3：1
整体风险判断：低。 这是一次结构良好的重构，将 device 构建与签名逻辑迁移至 cann-cmake 公共函数，同时升级依赖版本。唯一需要注意的是 --version CLI 选项的移除可能会影响已有的构建脚本，建议在发布说明中明确此项变更。
⚠️ 已识别出整体风险，但无法提取行内评论，请参考整体评估。
29 天前 添加了label：cann-cla/yes
CANN-robot
29 天前 评论：
29 天前 评论：
CANN-robot
29 天前 评论：
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 |
|---|---|---|
| CMakeLists.txt | ✅ 颜业峰, laodazhao1 (2/2) | ✅ 颜业峰 (1/1) |
| cmake | ✅ 颜业峰, laodazhao1 (2/2) | ✅ 颜业峰 (1/1) |
| docs | ✅ laodazhao1, 颜业峰 (2/2) | ✅ 颜业峰 (1/1) |
| repo-cann/hcomm | ✅ 颜业峰, laodazhao1 (2/2) | ✅ 颜业峰 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CANN-robot29 天前 将leabclove,yanglianxiang2,ccNeo824,flyswa,chengym,ccl_ligang,linyf950,yanzhenghang,chenliang267,claire8090,yanyefeng,chenhao_,dingweiqin_57,songmingyang,wenxuemin,yangxiaoyuan,laodazhao1,yangshaohua,chenke2026,zhongqin2020,bz_lee,Innoecho,chenyia,temper7620,zhangxp1030,abiggg,lilin_137,lilianlin_hw,hulk1997,ouyangxizi设为评审人
29 天前 将leabclove,yanglianxiang2,ccNeo824,flyswa,chengym,ccl_ligang,linyf950,yanzhenghang,chenliang267,claire8090,yanyefeng,chenhao_,dingweiqin_57,songmingyang,wenxuemin,yangxiaoyuan,laodazhao1,yangshaohua,chenke2026,zhongqin2020,bz_lee,Innoecho,chenyia,temper7620,zhangxp1030,abiggg,lilin_137,lilianlin_hw,hulk1997,ouyangxizi设为评审人
CANN-robot29 天前 将leabclove,yanglianxiang2,ccNeo824,flyswa,chengym,ccl_ligang,linyf950,yanzhenghang,chenliang267,claire8090,yanyefeng,chenhao_,dingweiqin_57,songmingyang,wenxuemin,yangxiaoyuan,laodazhao1,yangshaohua,chenke2026,zhongqin2020,bz_lee,Innoecho,chenyia,temper7620,zhangxp1030,abiggg,lilin_137,lilianlin_hw,hulk1997,ouyangxizi设为审查人
29 天前 将leabclove,yanglianxiang2,ccNeo824,flyswa,chengym,ccl_ligang,linyf950,yanzhenghang,chenliang267,claire8090,yanyefeng,chenhao_,dingweiqin_57,songmingyang,wenxuemin,yangxiaoyuan,laodazhao1,yangshaohua,chenke2026,zhongqin2020,bz_lee,Innoecho,chenyia,temper7620,zhangxp1030,abiggg,lilin_137,lilianlin_hw,hulk1997,ouyangxizi设为审查人
zhaoyingdong
29 天前 评论：
29 天前 评论：
/compile
Zzhaoyingdong
29 天前 update merge request[project id: 7874083, iid: 4176, commit_id: c17c6e5cb605f40edd2224bc70b4dd1d81553659] virtual merging success
29 天前 update merge request[project id: 7874083, iid: 4176, commit_id: c17c6e5cb605f40edd2224bc70b4dd1d81553659] virtual merging success
29 天前 添加了label：ci-pipeline-running
CANN-robot
29 天前 评论：
29 天前 评论：
流水线任务触发成功
任务链接 [4eadb65698034c3a861b6b26f12e52f8][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| SCA | ❌ FAILED | >>>>> | |
| antipoison | ✅ SUCCESS | >>>>> | |
| Check_Pr | ✅ SUCCESS | >>>>> | |
| Compile_Ascend_X86 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM | ✅ SUCCESS | >>>>> | >>>>> |
| pre_comment | ✅ SUCCESS | >>>>> | |
| Compile_Ascend_X86_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| StaticCheck_codespell | ✅ SUCCESS | ||
| StaticCheck_link_validity | ✅ SUCCESS | ||
| StaticCheck_resource_existence | ✅ SUCCESS | ||
| StaticCheck_tag_closed | ✅ SUCCESS | ||
| StaticCheck_markdownlint | ✅ SUCCESS |
29 天前 删除了label：ci-pipeline-running
29 天前 添加了label：ci-pipeline-failed
CANN-robot
29 天前 评论：
29 天前 评论：
流水线 PR-pipeline_cann_hcomm#16138 [ commitID：aef6548f ] 运行失败
zhaoyingdong
29 天前 评论：
29 天前 评论：
/compile
Zzhaoyingdong
29 天前 update merge request[project id: 7874083, iid: 4176, commit_id: de49111efdab8d5534b0f8def1f61c2a214152e9] virtual merging success
29 天前 update merge request[project id: 7874083, iid: 4176, commit_id: de49111efdab8d5534b0f8def1f61c2a214152e9] virtual merging success
29 天前 删除了label：ci-pipeline-failed
29 天前 添加了label：ci-pipeline-running
CANN-robot
29 天前 评论：
29 天前 评论：
流水线任务触发成功
任务链接 [107be08345574cbfa7daceedcca67bcd][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| SCA | ✅ SUCCESS | >>>>> | |
| antipoison | ✅ SUCCESS | >>>>> | |
| Check_Pr | ✅ SUCCESS | >>>>> | |
| Compile_Ascend_X86 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM | ✅ SUCCESS | >>>>> | >>>>> |
| pre_comment | ✅ SUCCESS | >>>>> | |
| Compile_Ascend_X86_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| StaticCheck_codespell | ✅ SUCCESS | ||
| StaticCheck_link_validity | ✅ SUCCESS | ||
| StaticCheck_resource_existence | ✅ SUCCESS | ||
| StaticCheck_tag_closed | ✅ SUCCESS | ||
| StaticCheck_markdownlint | ✅ SUCCESS | ||
| ST_Test | ✅ SUCCESS | ||
| API_Check | ✅ SUCCESS | >>>>> | |
| codecheck | ✅ SUCCESS | >>>>> | |
| codecheck_style | ✅ SUCCESS | >>>>> | |
| UT_Test | ✅ SUCCESS | >>>>> | |
| precommit | ✅ SUCCESS | >>>>> | |
| PreSmoke_A900_npupool | ✅ SUCCESS | >>>>> | |
| PreSmoke_A3_npupool | ✅ SUCCESS | >>>>> |
[2026-07-28 12:13:56] CI执行结束
29 天前 添加了label：api-check-pass
29 天前 删除了label：ci-pipeline-running
29 天前 添加了label：ci-pipeline-passed
CANN-robot
29 天前 评论：
29 天前 评论：
流水线 PR-pipeline_cann_hcomm#16150 [ commitID：aef6548f ] 已完成
laodazhao1
27 天前 评论：
27 天前 评论：
/lgtm
philo
27 天前 评论：
27 天前 评论：
/approve
27 天前 添加了label：lgtmapproved
27 天前 合入了pull request
描述
[refactor] 升级cann-cmake至master-044，迁移device构建与签名至cann-cmake公共函数
变更类型
请选择本次引入的变更类型：
关联的Issue
测试
已完成的测试用例和场景：
1.
2.
补充的UT用例：
文档更新
合入检查