Skip to content

  • 体验新版
    • 正在加载...
  • 登录
  • PaddlePaddle
  • Paddle
  • 合并请求
  • !27310

P
Paddle
  • 项目概览

PaddlePaddle / Paddle
大约 2 年 前同步成功

通知 2325
Star 20933
Fork 5424
  • 代码
    • 文件
    • 提交
    • 分支
    • Tags
    • 贡献者
    • 分支图
    • Diff
  • Issue 1423
    • 列表
    • 看板
    • 标记
    • 里程碑
  • 合并请求 543
  • Wiki 0
    • Wiki
  • 分析
    • 仓库
    • DevOps
  • 项目成员
  • Pages
P
Paddle
  • 项目概览
    • 项目概览
    • 详情
    • 发布
  • 仓库
    • 仓库
    • 文件
    • 提交
    • 分支
    • 标签
    • 贡献者
    • 分支图
    • 比较
  • Issue 1,423
    • Issue 1,423
    • 列表
    • 看板
    • 标记
    • 里程碑
  • 合并请求 543
    • 合并请求 543
  • Pages
  • 分析
    • 分析
    • 仓库分析
    • DevOps
  • Wiki 0
    • Wiki
  • 成员
    • 成员
  • 收起侧边栏
  • 动态
  • 分支图
  • 创建新Issue
  • 提交
  • Issue看板

Fix bug of handling blank characters in operators.cmake !27310

  • Report abuse
!27310 已合并 9月 15, 2020 由 saxon_zh@saxon_zh 创建
#<User:0x00007f0e6da3c5a0>
  • 概览 0
  • 提交 1
  • 变更 1

Created by: zhiqiu

PR types

Bug fixes

PR changes

Others

Describe

Fix bug of handling blank characters in operators.cmake

Paddle automatically add USE_OP(or USE_OP_DEVICE_KERNEL) for each operator target to pybind.h during running cmake (to avoid the operator registrar being removed from the generated binary file by the linker).

The approach is to extract REGISTER_OPERAOR( using regex for the source file of each operator, like xx_op.cc, and find the first registered operator.

However, the original implementation did not handle blank characters, which may result in errors, link https://github.com/PaddlePaddle/Paddle/pull/27112#issuecomment-691941000.

The problem is, due to code formatting, the REGISTER_OPERATOR(xxx may be separated into two lines. For example, elementwise_add_grad and elementwise_add_grad_grad in elementwise_add_op.cc.

REGISTER_OPERATOR(
    elementwise_add_grad, ops::ElementwiseOpGrad,
    ops::ElementwiseGradOpInplaceInferer, ops::ElementwiseGradNoBufVarsInferer,
    ops::ElementwiseAddDoubleGradMaker<paddle::framework::OpDesc>,
    ops::ElementwiseAddDoubleGradMaker<paddle::imperative::OpBase>);

REGISTER_OPERATOR(elementwise_add_grad_grad,
                  ops::ElementwiseOpDoubleGradWithoutDXDY,
                  ops::ElementwiseDoubleGradOpInplaceInferer,
                  ops::ElementwiseDoubleGradNoBufVarsInferer);

Since the REGISTER_OPERATOR(elementwise_add_grad, ...) is separated into two lines, the regex will match elementwise_add_grad_grad. And, since elementwise_add_mkldnn_op.cc is present, a line USE_OP_DEVICE_KERNEL(elementwise_add_grad_grad, MKLDNN) will be added to pybind.h, which results in https://github.com/PaddlePaddle/Paddle/pull/27112#issuecomment-691941000.

This PR fixes that.

指派人
分配到
审核者
Request review from
无
里程碑
无
分配里程碑
工时统计
标识: paddlepaddle/Paddle!27310
Source branch: github/fork/zhiqiu/dev/refine_operator_cmake
渝ICP备2023009037号

京公网安备11010502055752号

网络110报警服务 Powered by GitLab CE v13.7
开源知识
Git 入门 Pro Git 电子书 在线学 Git
Markdown 基础入门 IT 技术知识开源图谱
帮助
使用手册 反馈建议 博客
《GitCode 隐私声明》 《GitCode 服务条款》 关于GitCode
Powered by GitLab CE v13.7