已合并
【PR】: fix code alarm #2970
xiachanglin创建于 6月18日
【PR】: fix code alarm #2970
已合并
xiachanglin创建于 6月18日
已删除 :9.1.0合入到cann/runtime9.1.0
xiachanglin成员
6月18日

Pull Request

描述

告警清理。

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:
1.
2.

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 xiachanglin 的贡献)
Xxiachanglin成员
6月18日 添加了label:enhancement
Xxiachanglin成员
6月18日 创建了 pull request,commit b74d7b8d
CANN-robotCANN-robot成员
6月18日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
6月18日 评论:

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
src/inc/runtime Reyn52166, 侯延保 (2/2) Reyn52166 (1/1)
src/runtime Reyn52166, 侯延保 (2/2) Reyn52166 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
xiachanglin成员
6月18日 评论:

compile

likedislike
Xxiachanglin成员
6月18日 update merge request[project id: 8810053, iid: 2970, commit_id: 1ae5aba12c06c7d969eeb9d8115d65afd639ba32] virtual merging success
CANN-robotCANN-robot成员
6月18日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
6月18日 评论:

流水线任务触发成功
任务链接 [f7b706659440407b8b0a298ce909980f][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_acl ✅ SUCCESS >>>>>
UT_Test_rts_v201 ✅ SUCCESS >>>>>
UT_Test_rts_c ✅ SUCCESS >>>>>
UT_Test_rts_david ✅ SUCCESS >>>>>
UT_Test_rts_910b ✅ SUCCESS >>>>>
UT_Test_rts_common ✅ SUCCESS >>>>>
UT_Test_platform ✅ SUCCESS >>>>>
UT_Test_qs ✅ SUCCESS >>>>>
UT_Test_aicpusd ✅ SUCCESS >>>>>
UT_Test_tsd ✅ SUCCESS >>>>>
UT_Test_error_manager ✅ SUCCESS >>>>>
UT_Test_slog ✅ SUCCESS >>>>>
UT_Test_atrace ✅ SUCCESS >>>>>
UT_Test_msprof_part1 ✅ SUCCESS
UT_Test_msprof_part2 ✅ SUCCESS
UT_Test_adump ✅ SUCCESS >>>>>
UT_Test_mmpa ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>
UT_Test_rts_report ✅ SUCCESS >>>>>
UT_Test_msprof_report ✅ SUCCESS

[2026-06-18 20:25:32]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6月18日 评论:

流水线任务触发成功
任务链接 [f1b8ca522ff3462fb4dc362681a32adc][流水线指导]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Check_Pr ✅ SUCCESS
codecheck_style ✅ SUCCESS >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-06-18 20:22:05]    CI执行结束

likedislike
CANN-robotCANN-robot成员
6月18日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
6月18日 添加了label:ci-pipeline-passed
atomgit-bot
atomgit-bot
6月18日 评论:

我已经逐一审查了所有 10 个变更文件。这些改动均为编译器告警修复(如符号后缀不一致、const 修饰符位置、缺少 const、隐式类型转换等),未引入功能变更或逻辑缺陷。

变更摘要

此 PR 主要清理各类编译器告警,包括:为移位操作的右操作数添加 U 后缀以消除符号不匹配警告;为枚举常量十六进制字面量添加 U 后缀匹配底层无符号类型;在头文件声明处将 const 移至类型右侧(east-const 风格);为不修改成员变量的成员函数添加 const 限定;为不会被修改的局部变量添加 const 修饰;以及在 printf.ccprofiler.cc 中为 std::to_string 参数和三元表达式添加显式 static_cast 以消除隐式类型转换/缩窄告警。

主要改动

  • src/inc/runtime/rt_inner_device.h 枚举值移位操作统一 U 后缀: 将 rtAtomicOperationCapability 枚举中所有 1U << N 的移位右操作数从无符号整数改为带 U 后缀(如 1U << 0U),消除移位操作中符号不匹配的警告。
  • src/runtime/core/inc/printf.hpp 枚举值十六进制字面量添加 U 后缀: 将 DumpType(底层类型 uint32_t)的三个显式赋值常量 0xF0E00F0E 等改为 0xF0E00F0EU,匹配无符号枚举底层类型,消除符号警告。
  • src/runtime/core/inc/cond_handle/cond_handle.hpp 成员函数添加 const 限定: 为 GetCondType() 方法添加 const 限定符,表示该成员函数不修改对象状态,消除 const-correctness 相关告警。
  • src/runtime/core/src/dfx/printf.ccsrc/runtime/core/src/profiler/profiler.cc 显式类型转换: 在 PrintDumpTensor 中对 std::to_string 参数 tensorHead->positionuint16_t)添加 static_cast<uint32_t>;在 FillKernelLaunchExtInfo 中将 static_cast<uint16_t> 包裹整个三元表达式,消除隐式缩窄转换告警。
  • 其余文件 const 修饰与风格统一: 在 src/inc/runtime/rt_inner_mem.hsrc/runtime/api/api.hppsrc/runtime/core/inc_c/api_impl_david.hpp 将函数参数的 const 从类型左侧移至右侧(如 const T pT const p);在 src/runtime/core/src/device/device_error_proc.ccsrc/runtime/core/src/dfx/fp16_t.cpp 为不修改的局部变量添加 const。均为编译告警清理,无功能变更。
likedislike
atomgit-bot
atomgit-bot
6月18日 评论:

代码审查

✅ 未发现问题

likedislike
houyanbao成员
6月22日 评论:

/lgtm

likedislike
leihuan1成员
6月22日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
6月22日 添加了label:lgtmapproved
CANN-robotCANN-robot成员
6月22日 删除了label:ci-pipeline-passed
CANN-robot
CANN-robot成员
6月22日 评论:

The following label is not ready.

ci-pipeline-passed: The ci-pipeline-passed label is expired. Please compile again.

likedislike
xiachanglin成员
6月22日 评论:

compile

likedislike
Xxiachanglin成员
6月22日 update merge request[project id: 8810053, iid: 2970, commit_id: f92f91d967b03649bb9bc75d551c79c01eaf7398] virtual merging success
CANN-robotCANN-robot成员
6月22日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
6月22日 评论:

流水线任务触发成功
任务链接 [94cb1f90bf6940ec9a772b7eeefa0210][流水线指导]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_Check_Pr ✅ SUCCESS
codecheck_style ✅ SUCCESS >>>>>
codecheck_precommit ⚠️ WARNING >>>>> >>>>>
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS

[2026-06-22 09:24:44]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
6月22日 评论:

流水线任务触发成功
任务链接 [bafc26287da74ebe87f08ace4ecaab7e][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test_acl ✅ SUCCESS >>>>>
UT_Test_rts_v201 ✅ SUCCESS >>>>>
UT_Test_rts_c ✅ SUCCESS >>>>>
UT_Test_rts_david ✅ SUCCESS >>>>>
UT_Test_rts_910b ✅ SUCCESS >>>>>
UT_Test_rts_common ✅ SUCCESS >>>>>
UT_Test_platform ✅ SUCCESS >>>>>
UT_Test_qs ✅ SUCCESS >>>>>
UT_Test_aicpusd ✅ SUCCESS >>>>>
UT_Test_tsd ✅ SUCCESS >>>>>
UT_Test_error_manager ✅ SUCCESS >>>>>
UT_Test_slog ✅ SUCCESS >>>>>
UT_Test_atrace ✅ SUCCESS >>>>>
UT_Test_msprof_part1 ✅ SUCCESS
UT_Test_msprof_part2 ✅ SUCCESS
UT_Test_adump ✅ SUCCESS >>>>>
UT_Test_mmpa ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>
UT_Test_rts_report ✅ SUCCESS >>>>>
UT_Test_msprof_report ✅ SUCCESS

[2026-06-22 09:33:07]    CI执行结束

likedislike
CANN-robotCANN-robot成员
6月22日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
6月22日 添加了label:ci-pipeline-passed
CANN-robotCANN-robot成员
6月22日 合入了pull request