已合并
针对nn仓部分算子存在错误日志问题进行统一整改 #9248
gaomingyuan创建于 6 天前
针对nn仓部分算子存在错误日志问题进行统一整改 #9248
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 gaomingyuan 的贡献）6 天前 创建了 pull request，commit cd4648d6
atomgit-bot
6 天前 评论：
6 天前 评论：
变更摘要
本 PR 针对 nn 仓库中部分算子存在的错误日志问题进行统一整改，核心是对 activation 目录下多个算子（Elu、EluGrad、EluGradV2、FastGelu、FastGeluGrad、GeGluGradV2、GeGluV2）的 Tiling 代码进行日志规范化：删除大量冗余的 OP_LOGD 调试日志（函数进入/退出日志等），统一 OP_LOGE 错误日志的书写方式（使用 tilingContext->GetNodeName() 作为日志上下文并补充 outputDtype 信息），同时修正个别日志宏误用及格式问题，便于错误发生时定位问题。
主要改动
- 删除冗余调试日志：移除
elu_tiling_arch35.cpp、elu_grad_tiling_arch35.cpp、elu_grad_v2_tiling_arch35.cpp、ge_glu_grad_v2_tiling.cpp、ge_glu_v2_tiling.cpp等文件中CalcInputDtype、CheckShape、RunTiling、Tiling4GeGluV2等函数的入口/退出OP_LOGD日志，减少无业务意义的日志输出。 - 规范化
OP_LOGE错误日志：在Elu、EluGrad、EluGradV2、FastGelu、FastGeluGrad的RunTiling失败分支中，将OP_LOGE(tilingContext, ...)改为OP_LOGE(tilingContext->GetNodeName(), ...)，并新增ge::TypeUtils::DataTypeToSerialString输出outputDtype信息，使错误日志携带节点名与输出数据类型。 - 修复错误日志宏误用：
fast_gelu_grad_tiling_arch35.cpp中原本在失败分支误用OP_CHECK_NULL_WITH_CONTEXT打印 "elewiseBaseTiling failed" 日志，改为正确的OP_LOGE写法。 - 统一日志上下文来源：
ge_glu_grad_v2_tiling.cpp中Tiling4GeGluGradV2的错误日志由宏常量NODE_NAME改为context->GetNodeName()，保证日志上下文与实际节点一致。 - 修正格式问题：
ge_glu_grad_v2_tiling.cpp构造函数初始化列表{}前补充空格；elu、fast_gelu、fast_gelu_grad的 tiling 文件末尾补充缺失的换行符（新增 No newline at end of file）。
atomgit-bot
6 天前 评论：
6 天前 评论：
6 天前 添加了label：cann-cla/yes
CANN-robot
6 天前 评论：
6 天前 评论：
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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| activation | ✅ 汤平川, 王星 (2/2) | ✅ 汤平川, 王星 (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CANN-robot
6 天前 评论：
6 天前 评论：
CANN-robot6 天前 将zengjuan,chaotang233,kevin_huang1234,crystalhu,yangyang016,fanqirui,wangrui_,renruhai,su-yueming,zhajianqing123,chenqi317,wang-xing001,liubo75,tangweiwei2,liujie12345678,pingchuantang,Chen_HaoWen,qianzehong设为评审人
6 天前 将zengjuan,chaotang233,kevin_huang1234,crystalhu,yangyang016,fanqirui,wangrui_,renruhai,su-yueming,zhajianqing123,chenqi317,wang-xing001,liubo75,tangweiwei2,liujie12345678,pingchuantang,Chen_HaoWen,qianzehong设为评审人
CANN-robot6 天前 将kevin_huang1234,wangrui_,renruhai,su-yueming,zhajianqing123,wang-xing001,liujie12345678,pingchuantang,Chen_HaoWen,qianzehong设为审查人
6 天前 将kevin_huang1234,wangrui_,renruhai,su-yueming,zhajianqing123,wang-xing001,liujie12345678,pingchuantang,Chen_HaoWen,qianzehong设为审查人
6 天前 修改了pull request 的描述
gaomingyuan
6 天前 评论：
6 天前 评论：
compile
gaomingyuan6 天前 update merge request[project id: 7665709, iid: 9248, commit_id: c1f71101e46ab0d955ff900a8f648513676e32c1] virtual merging success
6 天前 update merge request[project id: 7665709, iid: 9248, commit_id: c1f71101e46ab0d955ff900a8f648513676e32c1] virtual merging success
6 天前 修改了pull request 的描述
6 天前 修改了pull request 的描述
gaomingyuan
6 天前 评论：
6 天前 评论：
compile
6 天前 添加了label：ci-pipeline-running
CANN-robot
6 天前 评论：
6 天前 评论：
流水线任务触发成功
任务链接 [dbd9570ab5b04bfdac9cb61d5f33c0eb][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| Compile_Ascend_X86_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_mobile_station | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_single | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_experimental | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_950 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_950 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Pre | ✅ SUCCESS | >>>>> | |
| pre_comment | ✅ SUCCESS | >>>>> | |
| Compile_Ascend_X86 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_mobile_station_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_X86_950_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_single_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_experimental_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_Ascend_ARM_950_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_910b | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_910c | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_X86_monitor_950 | ✅ SUCCESS | >>>>> | >>>>> |
| Compile_classify | ✅ SUCCESS | >>>>> | |
| Compile_Ascend_X86_mobile_station_9030_ubuntu24 | ✅ SUCCESS | >>>>> | >>>>> |
| UT_Test_ophost | ✅ SUCCESS | ||
| UT_Test_opapi | ✅ SUCCESS | ||
| UT_Test_kernel | ✅ SUCCESS | ||
| UT_Test_opgraph | ✅ SUCCESS | ||
| PreSmoke_A900 | ✅ SUCCESS | >>>>> | |
| API_Check | ✅ SUCCESS | >>>>> | |
| PreSmoke_ATK_Test_A2 | ✅ SUCCESS | >>>>> | |
| UT_Test_report_lcov | ✅ SUCCESS | >>>>> |
[2026-08-26 12:10:23] CI执行结束
CANN-robot
6 天前 评论：
6 天前 评论：
流水线任务触发成功
任务链接 [78e55b0747194d1fb91249651aa4e06f][流水线指导]
| 任务名称 | 状态 | 日志 | 下载链接 |
|---|---|---|---|
| antipoison | ✅ SUCCESS | >>>>> | |
| codecheck_checkpr | ✅ SUCCESS | ||
| codecheck_precommit | ✅ SUCCESS | >>>>> | |
| SCA | ✅ SUCCESS | >>>>> | |
| StaticCheck_md | ✅ SUCCESS |
[2026-08-26 11:46:48] CI执行结束
6 天前 添加了label：api-check-pass
6 天前 删除了label：ci-pipeline-running
6 天前 添加了label：ci-pipeline-passed
yuning_chen
6 天前 评论：
6 天前 评论：
您好，pr合入已准备就绪，请尽快联系committer进行检视，谢谢！
TangPC
6 天前 评论：
6 天前 评论：
/lgtm
/approve
6 天前 添加了label：approved
wang-xing001
6 天前 评论：
6 天前 评论：
/lgtm
/approve
6 天前 添加了label：lgtm
6 天前 合入了pull request
tangweiwei2
6 天前 评论：
6 天前 评论：
/lgtm
/approve
描述
本 PR 针对 nn 仓库中部分算子存在的错误日志问题进行统一整改，核心是对 activation 目录下多个算子（Elu、EluGrad、EluGradV2、FastGelu、FastGeluGrad、GeGluGradV2、GeGluV2）的 Tiling 代码进行日志规范化：删除大量冗余的 OP_LOGD 调试日志（函数进入/退出日志等），统一 OP_LOGE 错误日志的书写方式（使用 tilingContext->GetNodeName() 作为日志上下文并补充 outputDtype 信息），同时修正个别日志宏误用及格式问题，便于错误发生时定位问题。
关联的Issue
关联Issue#5183
测试
本地已完成新精度对比与性能测试，通过静态检查、蓝区冒烟、算子ST以及算子二级冒烟。
文档更新
类型标签
AI/Agent生成声明