| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
[build] replace public options with intf_pub of cann/cmake repo Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2529 merge fix-10 into master [build] replace public options with intf_pub of cann/cmake repo Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 启用 -Wextra 并修复了大部分编译警告,启用 -Werror 将编译警告转为编译错误,提升代码质量。 2. 构建系统变更通过add_cann_target_options集中管理编译选项,添加intf_pub链接,并移除冗余的针对每个目标的标志。 3. 警告修复解决了 -Wcast-function-type、-Wunused-function、-Wunused-variable、-Woverloaded-virtual、-Wimplicit-fallthrough、-Wtype-limit、-Wunused-command-line-argument、-Wsign-compare、-Wextra 和 -Wignored-qualifiers 等警告,覆盖80个文件。 还剩:-Wno-unused-parameter、-Wno-missing-field-initializers ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述:工程构建 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: - 编译成功 - 全量拦截任务通过 - 前冒烟测试通过 补充的UT用例: - NA ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2529 | 2 个月前 | |
[fix] the data race issue on variable oneSidedConns_ Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2572 merge issue-1 into master [fix] the data race issue on variable oneSidedConns_ Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 增加 RAII 锁解决单边通信变量 oneSidedConns_ 的数据竞争问题。 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> https://gitcode.com/cann/hcomm/issues/190 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. 编译成功 2. 全量拦截任务通过 3. 前冒烟测试通过 补充的UT用例: - **ascend910** **文件**: test/ut/framework/communicator/impl/one_sided_service/ut_hccl_one_sided_service.cc(新增 10 个) | # | 用例名 | 覆盖函数 | 覆盖分支 / 说明 | |---|---|---|---| | 1 | Ut_Prepare_When_PrepareFullMeshFailAndRankMissing_Expect_NoSpuriousEntry | Prepare | 失败清理:find()+.at() 即 break,不插入伪条目(**find/at 回归**) | | 2 | Ut_Prepare_When_AlreadyPrepared_Expect_ReturnSuccessImmediately | Prepare | prepared_ 为真直接返回 SUCCESS | | 3 | Ut_Prepare_When_TopoNotFullMesh_Expect_SkipFullMeshAndSuccess | Prepare | 非 FULLMESH 跳过建链,置 prepared_=true | | 4 | Ut_Prepare_When_PrepareFullMeshSuccess_Expect_PreparedTrue | Prepare | PrepareFullMesh 成功后 prepared_=true | | 5 | Ut_ExchangeMemDesc_When_SetupRemoteRankInfoFail_Expect_ReturnError | ExchangeMemDesc | 写路径 unique_lock + SetupRemoteRankInfo 失败,不入表 | | 6 | Ut_EnableMemAccess_When_ConnNotFound_Expect_ThrowLogicError | EnableMemAccess | 读路径 shared_lock + not-found 抛 logic_error | | 7 | Ut_DisableMemAccess_When_ConnNotFound_Expect_ThrowLogicError | DisableMemAccess | 读路径 shared_lock + not-found 抛 logic_error | | 8 | Ut_BatchPut_When_ConnNotFound_Expect_ThrowOutOfRange | BatchPut | 读路径 shared_lock + not-found 抛 out_of_range | | 9 | Ut_BatchGet_When_ConnNotFound_Expect_ThrowOutOfRange | BatchGet | 读路径 shared_lock + not-found 抛 out_of_range | | 10 | Ut_ConcurrentReadWrite_When_MixedReadWrite_Expect_NoDeadlock | ExchangeMemDesc + BatchPut | **多线程**:1 写(unique_lock) + 8 读(shared_lock),async+wait_for(10s) | - **ascend950** **文件**:test/legacy/ut/framework/service/one_sided_service/ut_hccl_one_sided_service.cc(新增 8 个) | # | 用例名 | 覆盖函数 | 覆盖分支 / 说明 | |---|---|---|---| | 1 | Ut_ExchangeMemDesc_When_AccStateNotAicpu_Expect_ReturnNotSupport | ExchangeMemDesc | accState 非 AICPU_TS 前置检查返回 HCCL_E_NOT_SUPPORT | | 2 | Ut_ExchangeMemDesc_When_CheckLinkFail_Expect_ReturnError | ExchangeMemDesc | 写路径 unique_lock + CheckLink 失败(RDMA→ROCE),不入表 | | 3 | Ut_ExchangeMemDesc_When_CreateConnectionFail_Expect_ReturnError | ExchangeMemDesc | 写路径 unique_lock + CheckLink 通过 + CreateConnection 失败,不入表 | | 4 | Ut_ExchangeMemDesc_When_ConnExists_Expect_Reuse | ExchangeMemDesc | 写路径 unique_lock + 已存在复用(else 分支),无重复插入 | | 5 | Ut_BatchPut_When_ConnNotFound_Expect_ThrowOutOfRange | BatchPut | 读路径 shared_lock + not-found 抛 out_of_range | | 6 | Ut_BatchGet_When_ConnNotFound_Expect_ThrowOutOfRange | BatchGet | 读路径 shared_lock + not-found 抛 out_of_range | | 7 | Ut_ConcurrentReads_When_MultiReadNotFound_Expect_NoDeadlock | BatchPut/BatchGet | **多线程**:8 读并发 shared_lock,async+wait_for(10s) | | 8 | Ut_ConcurrentReadWhileExchangeMemDesc_When_MixedReadWrite_Expect_NoDeadlock | ExchangeMemDesc + BatchPut | **多线程**:1 写(unique_lock) + 8 读(shared_lock),async+wait_for(10s) | ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论/lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2572 | 2 个月前 | |
[build] replace public options with intf_pub of cann/cmake repo Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2529 merge fix-10 into master [build] replace public options with intf_pub of cann/cmake repo Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 启用 -Wextra 并修复了大部分编译警告,启用 -Werror 将编译警告转为编译错误,提升代码质量。 2. 构建系统变更通过add_cann_target_options集中管理编译选项,添加intf_pub链接,并移除冗余的针对每个目标的标志。 3. 警告修复解决了 -Wcast-function-type、-Wunused-function、-Wunused-variable、-Woverloaded-virtual、-Wimplicit-fallthrough、-Wtype-limit、-Wunused-command-line-argument、-Wsign-compare、-Wextra 和 -Wignored-qualifiers 等警告,覆盖80个文件。 还剩:-Wno-unused-parameter、-Wno-missing-field-initializers ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述:工程构建 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: - 编译成功 - 全量拦截任务通过 - 前冒烟测试通过 补充的UT用例: - NA ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2529 | 2 个月前 | |
[build] replace public options with intf_pub of cann/cmake repo Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2529 merge fix-10 into master [build] replace public options with intf_pub of cann/cmake repo Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 启用 -Wextra 并修复了大部分编译警告,启用 -Werror 将编译警告转为编译错误,提升代码质量。 2. 构建系统变更通过add_cann_target_options集中管理编译选项,添加intf_pub链接,并移除冗余的针对每个目标的标志。 3. 警告修复解决了 -Wcast-function-type、-Wunused-function、-Wunused-variable、-Woverloaded-virtual、-Wimplicit-fallthrough、-Wtype-limit、-Wunused-command-line-argument、-Wsign-compare、-Wextra 和 -Wignored-qualifiers 等警告,覆盖80个文件。 还剩:-Wno-unused-parameter、-Wno-missing-field-initializers ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述:工程构建 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: - 编译成功 - 全量拦截任务通过 - 前冒烟测试通过 补充的UT用例: - NA ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2529 | 2 个月前 | |
[build] replace public options with intf_pub of cann/cmake repo Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2529 merge fix-10 into master [build] replace public options with intf_pub of cann/cmake repo Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 启用 -Wextra 并修复了大部分编译警告,启用 -Werror 将编译警告转为编译错误,提升代码质量。 2. 构建系统变更通过add_cann_target_options集中管理编译选项,添加intf_pub链接,并移除冗余的针对每个目标的标志。 3. 警告修复解决了 -Wcast-function-type、-Wunused-function、-Wunused-variable、-Woverloaded-virtual、-Wimplicit-fallthrough、-Wtype-limit、-Wunused-command-line-argument、-Wsign-compare、-Wextra 和 -Wignored-qualifiers 等警告,覆盖80个文件。 还剩:-Wno-unused-parameter、-Wno-missing-field-initializers ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述:工程构建 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: - 编译成功 - 全量拦截任务通过 - 前冒烟测试通过 补充的UT用例: - NA ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2529 | 2 个月前 | |
fix handle bug Co-authored-by: acoolntly<chenjianhui22@h-partners.com> # message auto-generated for no-merge-commit merge: !3342 merge master into master fix handle bug Created-by: acoolntly Commit-by: acoolntly Merged-by: cann-robot Description: ## 描述 新增 RdmaHandleManager::IsHandleValid 接口:在 rdma_handle_manager.h/cc 中新增 IsHandleValid(RdmaHandle) 公有方法,通过检查 activeHandles_ 集合、destroyed 标志和句柄非空来判断句柄是否仍然有效。 ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他,请描述: ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: 1. DS训练网络aicpu模式8p业务+device0卡注入OOM+0卡 2. 随机kill device侧进程 补充的UT用例: ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [ ] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论 /lgtm前的必要检查 - [ ] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [ ] 📄 代码修改内容已简要描述,相关文档已更新 - [ ] 📝 代码注释已更新,代码遵循项目整体代码风格 - [ ] 🧪 代码UT测试已更新,覆盖率已达标 - [ ] 🔬 验证方法已更新到"测试"部分 - [ ] 🛠️ 代码已通过静态分析工具检查,无错误 - [ ] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [ ] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [ ] 🔧 代码已通过compile,编译无错误,无告警 - [ ] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [ ] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3342 | 2 个月前 | |
[build] replace public options with intf_pub of cann/cmake repo Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2529 merge fix-10 into master [build] replace public options with intf_pub of cann/cmake repo Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 启用 -Wextra 并修复了大部分编译警告,启用 -Werror 将编译警告转为编译错误,提升代码质量。 2. 构建系统变更通过add_cann_target_options集中管理编译选项,添加intf_pub链接,并移除冗余的针对每个目标的标志。 3. 警告修复解决了 -Wcast-function-type、-Wunused-function、-Wunused-variable、-Woverloaded-virtual、-Wimplicit-fallthrough、-Wtype-limit、-Wunused-command-line-argument、-Wsign-compare、-Wextra 和 -Wignored-qualifiers 等警告,覆盖80个文件。 还剩:-Wno-unused-parameter、-Wno-missing-field-initializers ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述:工程构建 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: - 编译成功 - 全量拦截任务通过 - 前冒烟测试通过 补充的UT用例: - NA ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2529 | 2 个月前 | |
[build] replace public options with intf_pub of cann/cmake repo Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !2529 merge fix-10 into master [build] replace public options with intf_pub of cann/cmake repo Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 1. 启用 -Wextra 并修复了大部分编译警告,启用 -Werror 将编译警告转为编译错误,提升代码质量。 2. 构建系统变更通过add_cann_target_options集中管理编译选项,添加intf_pub链接,并移除冗余的针对每个目标的标志。 3. 警告修复解决了 -Wcast-function-type、-Wunused-function、-Wunused-variable、-Woverloaded-virtual、-Wimplicit-fallthrough、-Wtype-limit、-Wunused-command-line-argument、-Wsign-compare、-Wextra 和 -Wignored-qualifiers 等警告,覆盖80个文件。 还剩:-Wno-unused-parameter、-Wno-missing-field-initializers ## 变更类型 请选择本次引入的变更类型: <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他,请描述:工程构建 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue,可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景: - 编译成功 - 全量拦截任务通过 - 前冒烟测试通过 补充的UT用例: - NA ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前,请做好必要的代码测试,用例补充,软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签(如:[feat], [fix]) - [x] 📄 代码修改内容已简要描述,相关文档已更新 - [x] 📝 代码注释已更新,代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新,覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查,无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲,确保代码质量 - [x] ✅ 代码检视意见已处理或答复,无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile,编译无错误,无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试,确保基本功能正常 - [x] 🎯 预约 approver评论/approve,正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2529 | 2 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 2 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 2 个月前 |