已合并
fix:修复静态检查告警 #3936
Chang-an-HW创建于 7月9日
fix:修复静态检查告警 #3936
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 Chang-an-HW 的贡献)7月9日 创建了 pull request,commit c9c3cf40
atomgit-bot
7月9日 评论:
7月9日 评论:
变更摘要
该 PR 移除了 runtime/v1/graph/load/graph_loader.h 中不必要的 #include "rt_external_mem.h" 头文件引用。该引用源于此前重构提交中机械式地将 runtime/mem.h 替换为 rt_external_mem.h,但 graph_loader.h 及其实现文件并未使用该头文件提供的任何符号(如 rtMemType_t 等),删除该 include 不会影响任何下游文件的编译。
主要改动
- 移除冗余头文件引用: 在
runtime/v1/graph/load/graph_loader.h中删除了#include "rt_external_mem.h"一行,该头文件提供的符号在此文件中未被使用,所有依赖方已通过其他路径(如mem_manager.h、graph_var_manager.h)间接获得。


atomgit-bot
7月9日 评论:
7月9日 评论:
7月9日 添加了label:cann-cla/yes
CANN-robot
7月9日 评论:
7月9日 评论:
此处折叠了87条消息 查看更多
黄桂军
7月9日 评论:
7月9日 评论:
/lgtm


黄桂军
7月9日 评论:
7月9日 评论:
【MR评价】评价分数:3,
评价意见:
编码规范遵守度:无编码规范问题;
代码设计:不涉及代码设计;
DT质量:基本逻辑DT测试覆盖充足;


zhanj
7月9日 评论:
7月9日 评论:
/approve


7月9日 添加了label:approved
7月9日 合入了pull request
Pull Request
描述
修复静态检查告警
变更类型
关联的Issue
如何测试
核对清单
其他信息
该问题通过头文件依赖分析发现。
rt_external_mem.h是 CANN SDK 外部头文件,graph_loader.h中使用的所有类型(Status、ModelData、ModelParam、ModelQueueArg、ModelQueueParam、GeRootModelPtr、aclrtStream等)均由其他 include 提供。