已关闭
【pre-commit】clang-format应排除*_def.cpp算子定义文件(排版由开发者自行维护) #4848
景明创建于  6 天前关闭于  6 天前
景明
景明成员
6 天前 创建

问题描述

仓内 op_host/*_def.cpp 算子定义文件(如 mega_moe_def.cpp)包含大量 DataType({...})/Format({...}) 超长列表,开发者常按业务配置矩阵刻意对齐排版(尤其在 #ifdef 条件分支下)。clang-format 统一重排会破坏这种刻意的对齐布局,导致后续维护困难。

建议

在 pre-commit clang-format 钩子的 exclude 中增加 _def\.cpp$,排除全仓 222 个 *_def.cpp 文件的自动格式化,排版由开发者自行维护。

相关 PR #11041

likedislike
weihao18成员
6 天前 评论:

/assign @gitcode_lijd

likedislike
CANN-robotCANN-robot成员
6 天前 将 gitcode_lijd 设为负责人
景明景明成员
6 天前 修改标题为 “【pre-commit】clang-format应排除*_def.cpp及op_api/op_graph接口头文件(排版由开发者自行维护)”,原标题为“【pre-commit】clang-format应排除*_def.cpp算子定义文件(排版由开发者自行维护)”
景明景明成员
6 天前 修改了issue 的描述
景明景明成员
6 天前 修改标题为 “【pre-commit】clang-format应排除*_def.cpp算子定义文件(排版由开发者自行维护)”,原标题为“【pre-commit】clang-format应排除*_def.cpp及op_api/op_graph接口头文件(排版由开发者自行维护)”
景明景明成员
6 天前 修改了issue 的描述
CANN-robotCANN-robot成员
6 天前 关闭了 issue
CANN-robotCANN-robot成员
6 天前 添加了label:resolved