已合并
fix: 文件名重复问题解决,改引用公共目录下的文件 #9572
duxinlei创建于 5 天前
fix: 文件名重复问题解决,改引用公共目录下的文件 #9572
已合并
duxinlei创建于 5 天前
duxinlei成员
5 天前

描述

解决单仓内和多仓间文件名重复问题,
1、对于将公共文件引入单算子目录的重命名问题,改引用公共目录下的文件
2、对于单纯重名问题,重命名

关联的Issue

关联issue #5460

测试

david用例上板测试
david冒烟
obp冒烟

文档更新

类型标签

AI/Agent生成声明

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 duxinlei 的贡献)
Dduxinlei成员
5 天前 创建了 pull request,commit d914505c
atomgit-bot
atomgit-bot
5 天前 评论:

变更摘要

本 PR 旨在解决 pooling/avg_pool3_d 算子目录下文件与公共目录文件重名导致的问题:将 avg_pool3_d_tiling.cpp 中对 pool_tiling_templates_registry.h 的引用改为指向公共目录 pooling/pool_3d_common/op_host/arch35/ 下的同名文件,并删除算子目录下重复维护的本地副本 pool_tiling_templates_registry.htiling_base.h,使该算子统一复用公共目录中的 tiling 注册与基类实现,消除重复代码。

主要改动

  • 修改头文件引用路径avg_pool3_d_tiling.cpp 中将 #include "pool_tiling_templates_registry.h" 改为 #include "pooling/pool_3d_common/op_host/arch35/pool_tiling_templates_registry.h",改为引用公共目录下的文件。
  • 删除重复的 tiling 注册头文件:移除 pooling/avg_pool3_d/op_host/pool_tiling_templates_registry.h(169 行),其中包含 PoolTilingCasesPoolTilingRegistryPoolRegister 等类及 REGISTER_POOL_TILING_TEMPLATEREGISTER_OPS_POOL_TILING_TEMPLATE 宏定义。
  • 删除重复的 tiling 基类头文件:移除 pooling/avg_pool3_d/op_host/tiling_base.h(211 行),其中包含 TilingBaseClassAiCoreParams 等公共 tiling 基类定义,改由公共目录统一提供。
likedislike
atomgit-bot
atomgit-bot
5 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
5 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
5 天前 评论:

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
pooling 刘闯, 黄迪 (2/2) 黄迪, 刘闯 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了51条消息 查看更多
liuchuangdev成员
3 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
3 天前 添加了label:approved
kevin_huang1234成员
3 天前 评论:

/lgtm
/approve

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