已合并
refactor: simplify CMake configuration #205
zhangzijie创建于 4月27日
refactor: simplify CMake configuration #205
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 zhangzijie 的贡献）4月27日 创建了 pull request，commit 3777b82c
4月27日 添加了label：stat/needs-squash
CANN-robot
4月27日 评论：
4月27日 评论：
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 |
|---|---|---|
| repo-cann/cann-samples | ✅ 王林木, 周奇龙 (2/2) | ✅ 周奇龙, 王林木 (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
zhangzijie, thanks for your pull request. All authors of the commits have signed the CLA. 👍
4月27日 将yangyang016,llimwang设为评审人
4月27日 添加了label：cann-cla/yes
4月27日 将yangyang016,llimwang设为审查人
zhangzijie
4月27日 评论：
4月27日 评论：
compile
4月27日 添加了label：ci-pipeline-running
CANN-robot
4月27日 评论：
4月27日 评论：
4月27日 删除了label：ci-pipeline-running
4月27日 添加了label：ci-pipeline-passed
4月27日 添加了label：approved
cmake/shmem.cmake
已过期
@@ -34,7 +34,7 @@ add_custom_target(cann_samples_shmem_dependencies
3434 COMMAND ${CMAKE_COMMAND} -E env
3535 ASCEND_HOME_PATH=${ASCEND_DIR}
3636 ASCEND_TOOLKIT_HOME=${ASCEND_DIR}
37- cmake -S "${SHMEM_PATH}" -B "${SHMEM_PATH}/build" -DCMAKE_BUILD_TYPE=RELEASE -DSOC_TYPE=Ascend950
37+ cmake -S "${SHMEM_PATH}" -B "${SHMEM_PATH}/build" -DCMAKE_BUILD_TYPE=RELEASE -DSOC_TYPE=Ascend950 -DCMAKE_C_FLAGS=-w -DCMAKE_CXX_FLAGS=-w
3838 COMMAND ${CMAKE_COMMAND} --build "${SHMEM_PATH}/build" --target install --parallel
3939 WORKING_DIRECTORY "${PROJECT_SOURCE_DIR}"
4040 COMMENT "Initializing and building third_party/shmem"
@@ -75,6 +75,12 @@ target_link_libraries(cann_samples_shmem INTERFACE
7575 cann_samples_shmem_bootstrap_config_store
7676)
7777
78+target_compile_options(cann_samples_shmem INTERFACE
这里把 -w 作为 INTERFACE 编译选项透传给依赖方，会让上层目标（如 moe_dispatch_and_combine_story）的告警也被整体屏蔽，后续定位潜在问题会更难。建议仅在 third_party/shmem 子构建阶段保留 -w，不要通过接口库向外传播。
系统消息
4月27日 评论：
4月27日 评论：
4月27日 修改了pull request 的描述
Zzhangzijie
4月27日 update merge request[project id: 8788227, iid: 205, commit_id: 53af614ef4cba6ddb5122bb7ccb5c35aeeb48e43] virtual merging success
4月27日 update merge request[project id: 8788227, iid: 205, commit_id: 53af614ef4cba6ddb5122bb7ccb5c35aeeb48e43] virtual merging success
4月27日 推送 1 个提交:21ae5b45-Keep shmem warning suppression local
Zzhangzijie
4月27日 update merge request[project id: 8788227, iid: 205, commit_id: 3e1c90a5ab72f0edfbfb1dd32e743e4c44b9e74e] virtual merging success
4月27日 update merge request[project id: 8788227, iid: 205, commit_id: 3e1c90a5ab72f0edfbfb1dd32e743e4c44b9e74e] virtual merging success
4月27日 删除了label：ci-pipeline-passed
CANN-robot
4月27日 评论：
4月27日 评论：
4月27日 删除了label：approved
CANN-robot
4月27日 评论：
4月27日 评论：
zhangzijie
4月27日 评论：
4月27日 评论：
compile
4月27日 添加了label：ci-pipeline-running
CANN-robot
4月27日 评论：
4月27日 评论：
4月27日 添加了label：approved
4月27日 添加了label：lgtm
4月27日 删除了label：ci-pipeline-running
4月27日 添加了label：ci-pipeline-passed
CANN-robot
4月27日 评论：
4月27日 评论：
CANN-robot
4月27日 评论：
4月27日 评论：
The MR can not be merged, because of CodeReview discussion not resolved
If you want to solve this problem, you can click here to do it in the FAQs.
CANN-robot
4月27日 评论：
4月27日 评论：
CANN-robot
4月27日 评论：
4月27日 评论：
The MR can not be merged, because of CodeReview discussion not resolved
If you want to solve this problem, you can click here to do it in the FAQs.
4月27日 解决了最后一个问题
CANN-robot
4月27日 评论：
4月27日 评论：
4月27日 关闭了关联的issue
4月27日 合入了pull request
描述
简化样例工程的 CMake 配置，减少重复的 Ascend 基础库链接和冗余配置。
主要改动：
cmake/ascend.cmake和cmake/ascend_base.cmake。cmake/shmem.cmake中共享内存相关链接配置。关联的Issue
https://gitcode.com/cann/cann-samples/issues/132
测试
未运行自动化测试；本次改动集中在 CMake 配置清理。
文档更新
无需更新文档。
类型标签