已合并
[Profiling]修复runtime仓编译告警 #1405
zhengkai创建于 4月3日
[Profiling]修复runtime仓编译告警 #1405
已合并
zhengkai创建于 4月3日
zhengkai成员
4月3日

描述

  1. 修复4700059007U超过uint32_t最大值的告警,按照编译器规则取模,得到405091711U。
  2. 修复void FromObject(NanoJson::Json &object) override {}中object未使用的告警。
  3. 修复auto类型的i在判断大小时转换为int,为signed,但devIds.size()时size_t类型,为unsigned,两个类型不匹配的告警。
  4. 修复-fvisibility-inlines-hidden仅对CXX生效的告警。
  5. 修复类中成员定义顺序的告警。

关联的Issue

issue #218

测试

验证冒烟功能不受影响。

文档更新

NA

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhengkai 的贡献)
Zzhengkai成员
4月3日 创建了 pull request,commit 5a832b58
CANN-robot
CANN-robot成员
4月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
include 王涛, zhangpengpeng8 (2/2) 王涛 (1/1)
src/dfx zhuliangying, zhangpengpeng8 (2/2) zhuliangying (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
4月3日 添加了label:cann-cla/yes
CANN-robotCANN-robot成员
4月3日 将newstarzj,chenhao_1209,zhuliangying,wangtao43,zhangpengpeng8,yanmingxiang,Reyn52166,derekxu,tingwood设为评审人
CANN-robotCANN-robot成员
4月3日 将newstarzj,chenhao_1209,zhuliangying设为审查人
此处折叠了44条消息 查看更多
zhangpengpeng8成员
4月8日 评论:

/lgtm

likedislike
wangtao成员
4月8日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
4月8日 添加了label:lgtmapproved
CANN-robot
CANN-robot成员
4月8日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: zhuliangying, 王涛.
Reviewers who wrote a comment of /lgtm are: 王涛, zhangpengpeng8, zhuliangying.

likedislike
CANN-robotCANN-robot成员
4月8日 合入了pull request