已合并
fix: 修改文件名default拼写错误 #2471
fix: 修改文件名default拼写错误 #2471
已合并
wei_shi创建于 12 天前
wei_shi
wei_shi成员
12 天前

Pull Request

描述

修复 autofuse/ascir/reg_func/defalut_reg_func.h 文件名拼写错误,将 defalut 更正为 default。

autofuse/ascir/reg_func/ 目录下的头文件 defalut_reg_func.h 文件名存在拼写错误(default 误写为 defalut),且与同目录拼写正确的实现文件 default_reg_func.cpp 命名不一致,易造成混淆。本 PR 将该头文件重命名为 default_reg_func.h(文件内容不变),并同步更新全仓所有 #include 引用,共 41 个文件,+40/-40 行。

具体变更:

  • 文件重命名:autofuse/ascir/reg_func/defalut_reg_func.h → autofuse/ascir/reg_func/default_reg_func.h(similarity 100%,内容无变化)
  • 算子注册实现:更新 autofuse/ascir/reg_func/ 下 29 个算子实现文件(abs.cpp、broadcast.cpp、compare.cpp、split.cpp、where.cpp 等)的 #include "defalut_reg_func.h" → #include "default_reg_func.h"
  • codegen 生成器:更新 autofuse/ascir/generator/v1_ascir_codegen_impl.h 与 autofuse/v35/ascir/generator/v2_ascir_codegen_impl.h 中的引用
  • v35 v2 算子实现:更新 autofuse/v35/ascir/reg_func/ 下 exp2_v2.cpp、lgamma_v2.cpp、polygamma_v2.cpp、pow_v2.cpp 中的引用
  • 测试文件:更新 ST/UT 下 5 个引用该头文件的测试(test_reg_func_remainder.cpp、test_reg_func_split.cpp、test_reg_func_is_inf.cpp、test_reg_func_masked_fill.cpp 等)

不涉及接口、数据结构或运行逻辑的改动。

变更类型

关联的Issue

#223

如何测试

纯文件重命名 + include 路径同步修改,无功能变更,验证编译及无残留引用即可:

  1. 全仓搜索确认无残留引用:grep -rn "defalut_reg_func" autofuse/,结果为空
  2. 编译验证:sh build.sh --pkg -j 8 编译通过
  3. 编译受影响的 UT/ST 测试模块(autofuse_ascendc_api、autofuse_framework 等)通过

核对清单

其他信息

本 PR 为纯命名修正(rename similarity 100%),不影响任何运行逻辑,合入风险低。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 wei_shi 的贡献)
wei_shiwei_shi成员
12 天前 添加了label:enhancement
wei_shiwei_shi成员
12 天前 创建了 pull request,commit d4f23014
atomgit-bot
atomgit-bot
12 天前 评论:

变更摘要

本次变更是一次命名修正:将拼写错误的头文件 defalut_reg_func.h 重命名为正确的 default_reg_func.h(该文件本身内容未变,仅路径/文件名调整),并同步更新项目中所有引用该头文件的 #include 路径,涉及 autofuse 下的算子注册实现、代码生成器以及测试用例。变更不涉及接口、数据结构或运行逻辑的改动。

主要改动

  • 头文件重命名: autofuse/ascir/reg_func/defalut_reg_func.h 被重命名为 default_reg_func.h,修正 defalut 拼写错误。
  • 算子注册实现引用同步: abs.cpp、broadcast.cpp、div.cpp、reduce_max.cpp、split.cpp、welford.cpp、where.cpp 等 autofuse/ascir/reg_func 下的算子源文件将 #include "defalut_reg_func.h" 更新为 #include "default_reg_func.h"。
  • 代码生成器引用同步: v1_ascir_codegen_impl.h 与 v35/ascir/generator/v2_ascir_codegen_impl.h 中的相对路径引用改为 default_reg_func.h,保持生成器对注册函数的依赖一致。
  • v35 算子实现引用同步: exp2_v2.cpp、lgamma_v2.cpp、polygamma_v2.cpp、pow_v2.cpp 等 v2 算子实现改用 reg_func/default_reg_func.h。
  • 测试用例引用同步: st/ut/v35 下相关测试文件(如 test_reg_func_remainder.cpp、test_reg_func_split.cpp、test_reg_func_is_inf.cpp、test_reg_func_masked_fill.cpp)同步更新包含路径,避免因文件重命名导致编译失败。
likedislike
不准确?
atomgit-bot
atomgit-bot
12 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
12 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
12 天前 评论:

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
repo-cann/graph-autofusion ✅ yangyongqiang0606, xchu42, zhang_shengjie, wangxiaotian995 (4/2) ✅ yangyongqiang0606 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
12 天前 将niuyuhu,yangyongqiang0606,liyuewei,xchu42,zhang_shengjie,depeng1994,wqtshg_wt,peiyang,shengnan666,XuebinYang,sjtulxh,zhanj,zhujingjing,xuyafei,wangxiaotian995设为评审人
CANN-robotCANN-robot成员
12 天前 将niuyuhu,yangyongqiang0606,liyuewei,xchu42,zhang_shengjie,depeng1994,wqtshg_wt,peiyang,shengnan666,XuebinYang,sjtulxh,zhanj,zhujingjing,xuyafei,wangxiaotian995设为审查人
wei_shiwei_shi成员
12 天前 关联了issue:[Bug-Report|缺陷反馈]: 修复 autofuse/ascir/reg_func/defalut_reg_func.h 文件名拼写错误
wei_shiwei_shi成员
12 天前 修改了pull request 的描述
wei_shi
wei_shi成员
12 天前 评论:

compile

likedislike
wei_shiwei_shi成员
12 天前 预合并成功(commit_id: a84c04878e10eb0043855b09ea7594b63f47d951)
CANN-robot
CANN-robot成员
12 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
12 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
12 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
12 天前 添加了label:ci-pipeline-passed
xchu42
xchu42成员
12 天前 评论:

/lgtm

likedislike
zhang_shengjie成员
12 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
12 天前 添加了label:lgtm
wangxiaotian995成员
12 天前 评论:

/lgtm

likedislike
yangyongqiang
yangyongqiang成员
12 天前 评论:

/approve

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