已合并
【PR】: fix:整改魔鬼数字 #4053
【PR】: fix:整改魔鬼数字 #4053
已合并
duanpengliang创建于 15 天前
duanpengliang
duanpengliang成员
15 天前

Pull Request

描述

请清晰准确地描述本次 Pull Request 的意图和变更内容。

变更类型

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

关联的Issue

如何测试

跑测试用例覆盖日志输出看是否有魔鬼数字

核对清单

其他信息

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

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

变更摘要

本次 PR 主要对 ACL 和 Runtime 代码库中的日志输出进行系统性改进:将日志中直接打印枚举整数值(如 %d)改为调用枚举转字符串函数打印可读描述(格式为 NAME(value)UNKNOWN(value))。为此在 data_type_utils.h(ACL 侧)和 enum_desc.hpp / capture_model_enum_desc.hpp / task_enum_desc.hpp(Runtime 侧)新增了大量枚举描述函数,并修改了约 50 个文件中的日志调用点。同时部分已有枚举描述函数的标签前缀从 ACL_XXX(value) 简化为 XXX(value),去除了冗余的命名空间前缀。

主要改动

  • ACL 侧新增枚举描述函数: 在 src/acl/utils/data_type_utils.h 中新增 GetCannAttrDescGetDevResLimitTypeDescGetReduceKindDescGetMemLinkTypeDescGetErrorTypeDescGetConditionDescGetCompareDataTypeDescGetIpcMemAttrTypeDescGetFloatOverflowModeDescGetCmoTypeDescGetDeviceLimitDesc 共 11 个函数,统一返回 NAME(value)UNKNOWN(value) 格式字符串

  • Runtime 侧新增枚举描述函数: 在 src/runtime/core/inc/common/enum_desc.hpp 中新增 SysParamOptToStringDevResLimitTypeToStringConditionToStringKernelAttrTypeToStringLaunchKernelAttrIdToStringDevAttrToStringCmoOpCodeToStringMemLocationTypeToStringFloatOverflowModeToStringLimitTypeToStringMallocPolicyToStringMallocAttrToStringHacTypeToStringMemPoolAttrToStringDeviceStatusToStringDrvMemHandleTypeToStringMemSharedHandleTypeToStringManagedMemLocationTypeToString 等近 20 个函数

  • 已有枚举描述标签格式简化: data_type_utils.hGetSysParamOptDescGetMemLocationTypeDesc 等函数的返回标签从带 ACL_ 前缀(如 ACL_OPT_DETERMINISTIC(0))改为去除前缀(如 OPT_DETERMINISTIC(0));GetCallbackBlockTypeDesc 的期望值描述也同步更新

  • 约 50 个文件中的日志调用点更新: 将 ACL_LOG_ERROR/INFO/WARNRT_LOG 中的 %d(配合 static_cast<int32_t>(enumValue))全部替换为 %s(配合对应的枚举描述函数调用),覆盖 queue.cppdevice.cmemory.cacl.cppdevice.cppstream.cppapi_error.ccapi_impl.cc 等核心文件

  • 新增单元测试覆盖: 在 acl_common_unittest.cpprt_utest_enum_to_string_utils.cc 中分别新增测试用例验证已知枚举值的描述字符串格式以及未知枚举值返回 UNKNOWN(n) 的正确性

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

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
15 天前 添加了label:stat/needs-squash
此处折叠了238条消息 查看更多
duanpengliangduanpengliang成员
8 天前 修改了pull request 的描述
yanmingxiang成员
8 天前 评论:

/lgtm
/approve

likedislike
ykl999
ykl999成员
8 天前 评论:

/approve
/lgtm

likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:lgtmapproved
CANN-robotCANN-robot成员
8 天前 合入了pull request