| update tools clang format Co-authored-by: xun_zhuge<zhugexun@hisilicon.com> Co-authored-by: zhangyujia77<zhangyujia37@huawei.com> # message auto-generated for no-merge-commit merge: !3319 merge update_tools_clang_format into master update tools clang format Created-by: xun_zhuge Commit-by: zhangyujia77;xun_zhuge Merged-by: cann-robot Description: ## 描述 本 PR 将 tools/ 目录纳入 clang-format pre-commit 检查范围,并完成对应工具源码的格式治理、公共依赖收敛和 UT 覆盖率补充,降低后续工具目录代码风格漂移、重复代码和覆盖率门禁风险。 主要改动: - 更新 .pre-commit-config.yaml:移除 clang-format hook 对 tools/ 的排除,继续保留 tests/python/aclrt_launch_kernel/stub_files/、impl/、include/ 排除项。 - 对 tools/aclrtc、tools/build、tools/include 相关 C/C++ 源文件执行 clang-format,修复 tools 纳入扫描后的格式问题。 - 解除 tools/build/opbuild/common/ascendc_tool_log.h 重复头文件依赖,统一使用 tools/build/common/ascendc_tool_log.h,并同步 opbuild UT include 路径。 - 补充 tools 相关 UT 覆盖,包括 elf_tool、asc_pack_kernel、opbuild 等路径,提升 tools 目录覆盖率。 - 修复质量门禁问题:将 opbuild dtype 名称查询改为公共表驱动实现,避免大 switch case 导致圈复杂度超限,同时消除 op_cfg_generator.cpp 与 op_cpu_cfg_generator.cpp 间重复代码块。 ## 关联的Issue 无 ## 测试 已完成以下验证: - pre-commit validate-config 通过。 - pre-commit run clang-format --all-files 通过。 - 对本 PR 41 个新增/修改文件执行 bash scripts/oat_check.sh ...,OAT 增量扫描通过,日志显示 All checks passed (41 file(s) checked across 1 run(s))。 - 提交 hook 中 clang-format 与 OAT Compliance Check 均通过。 - 远端 ascend910b-smoke 干净工作区执行 ascendc_ut_asc_tikcpp_utest_opbuild:64/64 通过。 - 远端 ascend910b-smoke 执行 opbuild coverage:coverage.info_filtered 行覆盖率 92.1% (3563/3870)。 - 前序 tools UT coverage 验证:全量 tools filtered 行覆盖率 90.7% (4899/5399),本 PR executable tools 变更行覆盖率 95.9% (827/862)。 ## 文档更新 无文档更新。 ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 fix: Bug 修复 - [ ] ✨ feat: 新功能 - [ ] ⚡ perf: 性能优化 - [ ] ♻️ refactor: 代码重构 - [x] 🧪 test: 新增或修改测试 - [ ] 📝 docs: 文档更新 - [x] 🔧 ci: CI/CD 配置修改 - [ ] ↩️ revert: 回退 - [x] 🧹 chore: 其他,请具体描述 See merge request: cann/asc-devkit!3319 | 1 天前 |