Skip to content

  • 体验新版
    • 正在加载...
  • 登录
  • PaddlePaddle
  • Paddle
  • Issue
  • #9804

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看板
已关闭
开放中
Opened 4月 10, 2018 by saxon_zh@saxon_zhGuest

An ambiguous syntax in fluid/framework/dim.h

Created by: wangkuiyi

In the class template DimGetter, there are two signatures of impl:

https://github.com/PaddlePaddle/Paddle/blob/8dbd9c394e9cb97926320e113b112431ef509ec5/paddle/fluid/framework/dim.h#L109-L122

cpplint complains about the second impl because its parameter is not a const reference:

dim.h:118:  Is this a non-const reference? If so, make const or use a pointer: D& d  [runtime/references] [2]

This complaint intrigues us to think of the correct design:

  • The purpose of the first impl is to get, whereas the second is for *set, so

  • we should move the second impl into a new class template DimSetter:

    template <int i>
    struct DimSetter {
      HOSTDEVICE static void impl(const D&d, int64_t v) {
      ...
指派人
分配到
无
里程碑
无
分配里程碑
工时统计
无
截止日期
无
标识: paddlepaddle/Paddle#9804
渝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