修复aclnn代码生成接口的日志打印问题-asc-devkit-AtomGit

已合并

修复aclnn代码生成接口的日志打印问题 #4122

TvT989创建于 7月7日从weixin_62388497/asc-devkit_6444： 9.1.0 合入到 cann/asc-devkit： 9.1.0

修复aclnn代码生成接口的日志打印问题 #4122

已合并

TvT989创建于 7月7日

从weixin_62388497/asc-devkit_6444 9.1.0合入到cann/asc-devkit 9.1.0

TvT989

7月7日

描述

关联的Issue

测试

文档更新

类型标签

🐛 fix: Bug 修复
✨ feat: 新功能
⚡ perf: 性能优化
♻️ refactor: 代码重构
🧪 test: 新增或修改测试
📝 docs: 文档更新
🔧 ci: CI/CD 配置修改
↩️ revert: 回退
🧹 chore: 其他，请具体描述

Pull Request已成功合入, 合并人@CANN-robot

（感谢 TvT989 的贡献）

TvT989

7月7日创建了 pull request，commit 944cc137

atomgit-bot

7月7日评论：

变更摘要

此 PR 主要修复 aclnn 代码生成接口中与日志/错误报告相关的若干问题。核心改动包括：在 workspace 大小不足时引入结构化的错误报告机制（REPORT_PREDEFINED_ERR_MSG + NNOPBASE_INVALID_ARGUMENT），通过 needInvalidArgumentReport 参数有条件地启用无效参数报告的基础设施（宏定义和头文件引入），将 CheckAutoContiguousWarning 调用从 ValidateInputContiguousConflict 移至 GenerateCode 以避免重复检查，以及修正 ACLNN_OPTIONAL_PARAM_CHECK 环境变量的校验范围。此外还包含大量代码缩进与格式对齐的统一调整。

主要改动

Workspace 不足时的结构化错误报告：在 AclnnOpGenCodeRunUnContWithWorkspaceImpl 的 genContiguousCode lambda 中，将原来简单的 NnopbaseOpLogE 日志调用替换为 REPORT_PREDEFINED_ERR_MSG(NNOPBASE_INVALID_ARGUMENT, ...)，输出包含 workspaceSize、requiredSize 和 reason 的详细错误信息，便于问题定位。
有条件引入无效参数报告基础设施：AclnnGenCodeDecImpl、AclnnGenCheckInfo、AclnnGenCodeCommFunDelcare 三个函数新增 needInvalidArgumentReport 参数。当该参数为 true 时，AclnnGenCodeCommFunDelcare 会追加 #define NNOPBASE_INVALID_ARGUMENT "EZ1010" 宏，AclnnGenCodeDecImpl 会引入 <string>、<vector> 和 "base/err_msg.h" 头文件。AclnnGenCodeImplStart 中通过 AnalyzeSocAutoContiguousSupport 的结果决定该参数值。
CheckAutoContiguousWarning 调用位置调整：将该方法的调用从 ValidateInputContiguousConflict 内部移除，改为在 GenerateCode 主流程中直接调用。这避免了每次校验输入连续性冲突时都触发 AutoContiguous 警告检查，仅在实际生成代码阶段执行一次。
ACLNN_OPTIONAL_PARAM_CHECK 校验修复：引入常量 MAX_PARAM_CHECK_ENUM_NUM = 4UL，将校验条件从错误的 parsedVal > UINT32_MAX（该条件永远不成立）修正为 parsedVal >= MAX_PARAM_CHECK_ENUM_NUM，日志提示也相应更新为 "expected integer between [0, 3]"。

不准确？

atomgit-bot

7月7日评论：

代码审查

审查总结

本次审查覆盖了全部 5 个变更文件：

文件	审查结果
`tests/tools/utils/opbuild/aclnn_auto_contiguous_test.cpp.txt`	无问题（错误信息中的 "AutoContiguous API" 与该文件匹配）
`tests/tools/utils/opbuild/aclnn_ref_contiguous_test.cpp.txt`	P3: 错误信息中 API 名称复制粘贴错误
`tests/tools/utils/opbuild/aclnn_rms_norm_grad_test.cpp.txt`	P3: 错误信息中 API 名称复制粘贴错误
`tests/tools/utils/opbuild/aclnn_soc_version_dynamic_ignore_cont_test.cpp.txt`	P3: 错误信息中 API 名称复制粘贴错误
`tests/tools/utils/opbuild/aclnn_soc_version_test1.cpp.txt`	P3: 错误信息中 API 名称复制粘贴错误

统计

P0: 0
P1: 0
P2: 0
P3: 1（同一问题的 4 个文件实例，合并为一个 finding）

总体风险判断

低风险。本次变更将旧的简单日志调用 NnopbaseOpLogE 替换为更详细的 REPORT_PREDEFINED_ERR_MSG 错误上报机制，这是对日志质量的改进。核心逻辑（条件判断、错误返回码）未发生变化，不存在正确性、安全性或可靠性风险。

唯一发现的问题是错误信息中的 API 名称存在复制粘贴错误：在 4 个非 AutoContiguous 测试文件中，reason 字符串均引用了 "AutoContiguous API"，与各自函数的实际 API 名称不一致，可能对排查问题的用户造成困惑。由于全部变更位于测试类文件（.cpp.txt），对生产环境无直接影响。

⚠️ 已识别出整体风险，但无法提取行内评论，请参考整体评估。

不准确？

任务名称	状态	日志	下载链接
SCA	✅ SUCCESS	>>>>>
antipoison	✅ SUCCESS	>>>>>
codecheck	✅ SUCCESS	>>>>>
codecheck_Pr	✅ SUCCESS
pre_comment	✅ SUCCESS	>>>>>
StaticCheck_codespell_check	✅ SUCCESS
StaticCheck_link_validity_check	✅ SUCCESS
StaticCheck_resource_existence_check	✅ SUCCESS
StaticCheck_tag_closed_check	✅ SUCCESS
StaticCheck_markdownlint	✅ SUCCESS
codecheck_codestyle	✅ SUCCESS	>>>>>
codecheck_precommit	✅ SUCCESS	>>>>>	>>>>>

任务名称	状态	日志	下载链接
Compile_Ascend_X86	✅ SUCCESS	>>>>>	>>>>>
Compile_Ascend_ARM	✅ SUCCESS	>>>>>	>>>>>
pre_comment	✅ SUCCESS	>>>>>
PreSmoke_A900_npupool	✅ SUCCESS		>>>>>

module	committer
adv_api	chenyiyuan, wuyang_hw
basic_api	chenyiyuan, wuyang_hw
c_api	pangfd, yue_mo_du_qian
cmake	xun_zhuge, DragonBornHD84
docs	munanhw, ai_xin
examples	changxianyu, bianque
simt_api	wulinyu, zhanghao_0689
scripts	xun_zhuge, kevin365
tools	YANXI_ZHAO, XuebinYang, sjtulxh

描述