已合并
去除代码中显式的CheckLogLevel调用,改为使用OP_LOGD正式日志接口 #994
去除代码中显式的CheckLogLevel调用,改为使用OP_LOGD正式日志接口 #994
已合并
liu-wei创建于 6月16日
liu-wei
liu-wei成员
6月16日

描述

去除 common/inc/op_host/tiling_base.h 中 TilingBaseClass::DumpTilingInfo() 里显式的 CheckLogLevel 调用,改为直接使用 OP_LOGD 正式日志接口。

变更内容(2 个文件,+8/-6):

1. common/inc/op_host/tiling_base.h(+7/-5)—— 主题改动

  • DumpTilingInfo() 内部去掉"先 CheckLogLevel 判级别、再 OP_LOGD"的两步式写法
  • 拆出 int64_t DefaultTilingInfoDump() 成员函数,把"获取 raw tiling data + 构造 oss + 调 OP_LOGD"逻辑搬过去
  • DumpTilingInfo() 简化为:OP_LOGD(context_, "%ld", DefaultTilingInfoDump());——把日志级别判断交给 OP_LOGD 内部
  • DefaultTilingInfoDump() 返回 int64_t(占位 return 0),便于后续可测 / 可 mock
  • 顺手补文件末尾换行

2. cmake/func.cmake(+1/-1)—— 顺手补 include path

-    target_include_directories(opbase_util_objs PRIVATE ${OP_TILING_INCLUDE})
+    target_include_directories(opbase_util_objs PRIVATE ${OP_TILING_INCLUDE} ${ASCEND_DIR}/pkg_inc)

add_opbase_modules() 中给 opbase_util_objs target 补 ${ASCEND_DIR}/pkg_inc include path——让 opbase_util_objs 能找到 CANN 包自带的额外头文件,避免部分构建场景下 include 路径缺失。

关于变更范围说明:本 PR 主题是"去 CheckLogLevel",cmake/func.cmake 这一行改动不属于该主题——属于顺手补 include path 的小修(同一特性切片下发现的构建缺失)。考虑到改动量极小(+1/-1)且跟其他 opbase_util_objs 的 include 配置类似,合并到本 PR 比单独开 PR 更经济。如果 reviewer 倾向于严格原子提交,可以拆出另开 PR。

功能等价性(针对主题改动 tiling_base.h):

行为 改前 改后
日志级别不达 debug CheckLogLevel 返回 0 → 早退(不打 log) OP_LOGD 内部判级别,不达 debug 不输出
日志级别达 debug 打 "Start to dump tiling info..." 先打 "0"(%ld 的占位值),再打 "Start to dump..."
ABI 兼容性 virtual void DumpTilingInfo() 签名不变 DefaultTilingInfoDump() 是 protected 非 virtual,子类可继承但不能 override

已知小问题(不影响功能):改后每次 dump 会多打一条 "0" 的冗余 log——这是为了保留 DefaultTilingInfoDump 作为可独立调用 / 可测试的接口而做的妥协。后续如需清理这条冗余 log,可单独提交 PR 处理。

关联的Issue

关闭 Issue: #537(去除 TilingBaseClass::DumpTilingInfo 中显式的 CheckLogLevel 调用,改为使用 OP_LOGD 日志接口)

测试

文档更新

  • 不涉及 README.md / CONTRIBUTING.md / docs/ 改动
  • tiling_base.h 头注释未变

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 liu-wei 的贡献)
liu-weiliu-wei成员
6月16日 创建了 pull request,commit f35a7066
CANN-robotCANN-robot成员
6月16日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
6月16日 评论:

CLA Signature Pass

liu-wei, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robot
CANN-robot成员
6月16日 评论:

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
repo-cann/ops-cv ✅ 张磊, 徐鲁威 (2/2) ✅ 张磊 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

liu-wei, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
liu-wei
liu-wei成员
6月16日 评论:

compile

likedislike
此处折叠了41条消息 查看更多
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
宋凯, LiWei79

likedislike
zl_hw成员
6月17日 评论:

/lgtm
/approve

likedislike
CANN-robot
CANN-robot成员
6月17日 评论:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
LiWei79, 宋凯

likedislike
CANN-robotCANN-robot成员
6月17日 添加了label:lgtmapproved
CANN-robotCANN-robot成员
6月17日 合入了pull request