已合并
fix: UT门禁拦截 #127
哈喽Kiter创建于 8月21日
fix: UT门禁拦截 #127
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 哈喽Kiter 的贡献)atomgit-bot
8月21日 评论:
8月21日 评论:
变更摘要
该 PR 为测试相关的改动,仅修改了 include/aicore/ain/ain.h 中的一处头文件包含路径:将 ain_impl_def.h 的引用方式由相对路径 ../../../../impl/comm_api/aicore/ain/impl/ain_impl_def.h 改为以根目录开头的路径 /impl/comm_api/aicore/ain/impl/ain_impl_def.h,改动本身不影响接口定义或业务逻辑,仅涉及编译期头文件解析方式。
主要改动
- 头文件包含路径调整: 在
include/aicore/ain/ain.h中,#include "ain_impl_def.h"的路径由相对路径../../../../impl/comm_api/aicore/ain/impl/ain_impl_def.h改为绝对路径形式/impl/comm_api/aicore/ain/impl/ain_impl_def.h,影响该头文件被引用时的编译查找行为。


不准确?
atomgit-bot
8月21日 评论:
8月21日 评论:
代码审查
关闭摘要
变更文件审查确认
- include/aicore/ain/ain.h — 已审查(发现 1 个问题,见下)
发现统计
- P0:1 个
- P1:0 个
- P2:0 个
- P3:0 个
总体风险判断
本 PR 唯一的改动(ain.h 第 25 行)将 ain_impl_def.h 的包含路径从仓库相对路径 ../../../../impl/... 改写为以 / 开头的绝对路径 /impl/...。该路径按文件系统根目录解析,无法命中仓库内的目标头文件,任何包含 ain.h 的翻译单元都会在预处理阶段直接报 No such file or directory 而编译失败,属于确定的构建破坏(P0)。同文件第 225 行对同类头文件 ain_impl.h 仍使用相对路径,进一步印证原路径才是仓库布局下的正确写法。建议在合入前恢复原相对路径。
| 类型 | 数量 |
|---|---|
| 🔴 阻塞 | 1 |
| 🟡 建议 | 0 |
⛔ 需要修改


不准确?
哈喽Kiter
8月21日 评论:
8月21日 评论:
compile


8月21日 添加了label:cann-cla/yes
此处折叠了122条消息 查看更多
哈喽Kiter
8月24日 评论:
8月24日 评论:
/check-pr


CANN-robot
8月24日 评论:
8月24日 评论:
The following labels are not ready.
lgtm: Please wait for reviewers to review the code.
approved: Please wait for committers to review the code.


izanaami
8月24日 评论:
8月24日 评论:
/approve


8月24日 添加了label:lgtmapproved
8月24日 合入了pull request
当前PR是否有AI参与:
[x] 否
[ ] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :
PR功能描述 / 为什么需要这个合入**:
修复 UT 门禁拦截失效问题
该PR关联的issue
(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #
希望检视人员了解:
改动类型 / Change Type
测试信息 / Testing
检查清单 / Checklist