已合并
fix: 修复多算子代码告警问题 #3784
fix: 修复多算子代码告警问题 #3784
已合并
yuanbin_22创建于 7月3日
yuanbin_22成员
7月3日

描述

fix: 修复多算子代码告警问题

关联的Issue

https://gitcode.com/cann/ops-nn/issues/1323

测试

文档更新

类型标签

  • Bug修复
  • 新特性
  • 性能优化
  • 文档更新
  • 其他,请描述:
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 yuanbin_22 的贡献)
Yyuanbin_22成员
7月3日 创建了 pull request,commit 62b6dfc2
CANN-robotCANN-robot成员
7月3日 添加了label:cann-cla/no
CANN-robot
CANN-robot成员
7月3日 评论:

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


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
math/abs 周奇龙, 宋恺 (2/2) 周奇龙, 宋恺 (2/1)
math/cast 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/ceil 宋恺, 周奇龙 (2/2) 周奇龙, 宋恺 (2/1)
math/cos 周奇龙, 宋恺 (2/2) 宋恺, 周奇龙 (2/1)
math/exp 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/floor 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/is_finite 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/is_inf 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/is_neg_inf 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/is_pos_inf 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/log 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/masked_scale 周奇龙, 宋恺 (2/2) 宋恺, 周奇龙 (2/1)
math/neg 周奇龙, 宋恺 (2/2) 宋恺, 周奇龙 (2/1)
math/reciprocal 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/round 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/rsqrt 宋恺, 周奇龙 (2/2) 周奇龙, 宋恺 (2/1)
math/sign 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/sin 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/sqrt 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
math/square 宋恺, 周奇龙 (2/2) 周奇龙, 宋恺 (2/1)
math/tanh 宋恺, 周奇龙 (2/2) 周奇龙, 宋恺 (2/1)
math/trunc 宋恺, 周奇龙 (2/2) 宋恺, 周奇龙 (2/1)
repo-cann/ops-math 宋恺, 周奇龙 (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

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

likedislike
Yyuanbin_22成员
7月3日 修改了pull request 的描述
atomgit-bot
atomgit-bot
7月3日 评论:

变更摘要

此 PR 主要修复多个算子代码中的编译器告警问题,涉及两类改动:为结构体成员变量补充默认初始值以消除"未初始化变量"告警,以及将文件作用域的 using namespace 声明移入更窄的命名空间作用域以避免全局命名空间污染。改动覆盖 math/ 目录下 castceilis_finiteis_infis_nanis_neg_infis_pos_infmasked_scaleroundrsqrtsinsqrttanhtrunc 共 14 个算子的头文件。

主要改动

  • 结构体成员默认初始化: 为 CastCompileInfoCeilCompileInfoIsFiniteTilingDataIsNanCompileInfoIsNegInfCompileInfoIsPosInfCompileInfoMaskedScaleCompileInfoRoundCompileInfoRsqrtCompileInfoSqrtCompileInfo 等结构体中的整型/浮点型成员变量添加 = 0(或 = 0.0f= ge::DT_UNDEFINED)默认值,消除未初始化变量告警。

  • CastMapSt 结构体字段默认初始化: 为 CastMapSt 中的 srcType_dstType_ 设为 ge::DT_UNDEFINED,其余 uint8_t 类型字段(id_srcMapType_dstMapType_midType_castMode1_castMode2_regCopyInMode_regCopyOutMode_)设为 0,避免未定义行为。

  • ge::DataType 成员默认值: 在 is_finiteis_inf 的 tiling 头文件中,将 inputDtypeoutputDtype 字段初始化为 ge::DT_UNDEFINED,防止未初始化的枚举值被使用。

  • using namespace 作用域收敛: 将 sin_tiling_struct.htrunc_tilingdata.h 中文件级的 using namespace Ops::Base 移除,改为在结构体定义内使用完全限定名 Ops::Base::EleBaseTilingData;在 tanh_dag.h 中,将 using namespace Ops::Baseusing namespace AscendC 从文件作用域移入 TanhDag1 命名空间内,同时将 DAG 类型定义中的 BindVec::CopyInPlaceholder 等改为完全限定名,消除全局命名空间污染告警。

likedislike
此处折叠了269条消息 查看更多
CANN-robotCANN-robot成员
21 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
21 天前 添加了label:ci-pipeline-passed
songkai111成员
20 天前 评论:

/approve

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