已合并
【PR】: mv core/api_impl->api/impl #3999
duanpengliang创建于 26 天前
【PR】: mv core/api_impl->api/impl #3999
已合并
duanpengliang创建于 26 天前
duanpengliang
duanpengliang成员
26 天前

Pull Request

描述

把core/api_impl移动到api/impl,同时修复根据驱动生成的桩.so缺失符号问题

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:
1.
2.

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 duanpengliang 的贡献)
duanpengliangduanpengliang成员
26 天前 添加了label:enhancement
duanpengliangduanpengliang成员
26 天前 创建了 pull request,commit 4a1c28e6
atomgit-bot
atomgit-bot
26 天前 评论:

变更摘要

该 PR 是一个纯粹的目录重构变更:将 api_impl 相关源文件从 src/runtime/core/src/api_impl/ 整体迁移至 src/runtime/api/impl/,并同步更新了 6 个 CMake 构建脚本中所有的源文件路径引用和头文件包含路径。涉及 31 个文件的目录移动(内容不变)以及 6 个 .cmake 文件的路径修改,净变更为代码行增删持平。

主要改动

  • api_impl 目录迁移:将 src/runtime/core/src/api_impl/ 下的所有实现文件(包括 api_decoratorapi_errorapi_implapi_impl_creatorapi_impl_mbuf 等)及其子目录 v100/v200/v201/ 整体移至 src/runtime/api/impl/,文件内容不变。

  • CMake 源文件路径批量更新:在 arch5162.cmakecmodel.cmakeplatform.cmakeruntime.cmaketiny.cmakev200.cmake 六个构建脚本中,将所有 ${RUNTIME_CORE_DIR}/src/api_impl/... 源文件路径统一替换为 ${RUNTIME_DIR}/src/runtime/api/impl/...

  • 头文件包含路径调整:在各 CMake 文件中的 RUNTIME_INC_DIR_* 变量里,将旧的 ${RUNTIME_CORE_DIR}/src/api_impl 包含路径替换为 ${RUNTIME_DIR}/src/runtime/api/impl,确保编译时头文件查找正确。

likedislike
atomgit-bot
atomgit-bot
26 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
26 天前 添加了label:cann-cla/yes
此处折叠了75条消息 查看更多
ykl999
ykl999成员
12 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
12 天前 添加了label:lgtm
CANN-robot
CANN-robot成员
12 天前 评论:

The MR can not be merged, because of CodeReview discussion not resolved

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
duanpengliangduanpengliang成员
12 天前 解决了最后一个问题
CANN-robotCANN-robot成员
12 天前 合入了pull request