Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

【编译优化】paddle_test 推全 #60793

Closed
Galaxy1458 opened this issue Jan 15, 2024 · 6 comments
Closed

【编译优化】paddle_test 推全 #60793

Galaxy1458 opened this issue Jan 15, 2024 · 6 comments
Assignees
Labels
HappyOpenSource 快乐开源活动issue与PR status/close 已关闭 type/others 其他问题

Comments

@Galaxy1458
Copy link
Contributor

Galaxy1458 commented Jan 15, 2024

问题描述 Please describe your issue

一、需求背景

飞桨正在做编译优化工作,此需求为 [cmake治理]删除paddle_test多余无用依赖 的升级版需求,背景原理类似,目前paddle编译c++单测有多种方式,如cc_test(windows下link静态库,linux下link paddle.so),paddle_test(windows和linux下都link动态库paddle.so),现在推荐使用paddle_test,好处在于:

  1. link动态库无需手动加过多的依赖,大部分符号在paddle.so中已存在;
  2. link libpaddle.so可以大幅度减小C++单测的体积 ,从而减少build目录体积大小

二、存在的问题

paddle仓库中有43个使用cc_test的CMakeLists.txt文件,其对paddle的单测编译造成一定压力,现打算将其全部替换为paddle_test.

三、建议步骤

在包含较多cc_test的单个CMakeLists.txt里建议拆分处理。

可能遇到的编译问题:

请参考:https://github.com/PaddlePaddle/community/blob/master/pfcc/call-for-contributions/code_style/change_cc_test_to_paddle_test.md

四、任务划分

序号 文件名 认领人/状态/PR号
1 test/cpp/pir/pattern_rewrite/CMakeLists.txt @Liyulingyue #60830
2 test/cpp/pir/pass/CMakeLists.txt @Liyulingyue #60831
3 test/cpp/pir/kernel_dialect/CMakeLists.txt @Liyulingyue #60832
4 test/cpp/pir/core/CMakeLists.txt @Liyulingyue #60833
5 test/cpp/pir/control_flow_dialect/CMakeLists.txt @Liyulingyue #60834
6 test/cpp/phi/ops/CMakeLists.txt @DrRyanHuang #60919
@Liyulingyue #61985
7 test/cpp/phi/kernels/CMakeLists.txt @DrRyanHuang #60917
@Liyulingyue #61984
8 test/cpp/phi/core/CMakeLists.txt @DrRyanHuang #60918
@Liyulingyue #61980
9 test/cpp/phi/common/CMakeLists.txt @DrRyanHuang #60916
@Liyulingyue #61981
10 test/cpp/phi/api/CMakeLists.txt @DrRyanHuang #60915
@Liyulingyue #61983
11 test/cpp/new_executor/CMakeLists.txt @zade23 #61013
12 test/cpp/jit/CMakeLists.txt @zade23 #61014
13 test/cpp/inference/api/CMakeLists.txt @zade23 #61016
14 test/cpp/imperative/CMakeLists.txt @zade23 #61017
15 test/cpp/fluid/reader/CMakeLists.txt @zade23 #61018
16 test/cpp/fluid/mkldnn/CMakeLists.txt @Liyulingyue #61672
17 test/cpp/fluid/memory/CMakeLists.txt @gouzil #61105
@Liyulingyue #62031
18 test/cpp/fluid/math/CMakeLists.txt @Liyulingyue #61673
19 test/cpp/fluid/fused/CMakeLists.txt @ccsuzzh #61238
20 test/cpp/fluid/framework/details/CMakeLists.txt @Liyulingyue #61463
21 test/cpp/fluid/framework/CMakeLists.txt @Liyulingyue #61674 #62534
22 test/cpp/fluid/elementwise/CMakeLists.txt @Liyulingyue #61376
23 test/cpp/fluid/dlnne/CMakeLists.txt @Liyulingyue #61372
24 test/cpp/fluid/detection/CMakeLists.txt @Liyulingyue #61373
25 test/cpp/fluid/controlflow/CMakeLists.txt @Liyulingyue #61374
26 test/cpp/fluid/benchmark/CMakeLists.txt @Liyulingyue #61375
27 test/cpp/fluid/CMakeLists.txt @Liyulingyue #61675
28 test/cpp/eager/task_tests/CMakeLists.txt @Liyulingyue #61676
29 test/cpp/eager/data_structure_tests/CMakeLists.txt @Liyulingyue #61677
30 test/cpp/auto_parallel/CMakeLists.txt @Liyulingyue #61462
31 paddle/utils/string/CMakeLists.txt @xiaoyewww #60958 #60959
32 paddle/utils/CMakeLists.txt @xiaoyewww #60959
33 paddle/phi/core/distributed/store/CMakeLists.txt @xiaoyewww #60960
34 paddle/phi/backends/CMakeLists.txt @xiaoyewww #60961
35 paddle/fluid/platform/profiler/CMakeLists.txt @ooooo-create #61093
@Liyulingyue #62034 #62033
36 paddle/fluid/platform/device/custom/CMakeLists.txt @ooooo-create #61094
@Liyulingyue #62036 #62033
37 paddle/fluid/platform/CMakeLists.txt @ooooo-create #61095
@Liyulingyue #62035 #62033
38 paddle/fluid/inference/api/details/CMakeLists.txt @ooooo-create
@Liyulingyue #61943
39 paddle/fluid/inference/analysis/ir_passes/CMakeLists.txt @ooooo-create
@Liyulingyue #61944
40 paddle/fluid/framework/ir/memory_optimize_pass/CMakeLists.txt @ooooo-create
@Liyulingyue #61945
41 paddle/fluid/framework/ir/fusion_group/CMakeLists.txt @ooooo-create
@Liyulingyue #61946
42 paddle/fluid/framework/ir/CMakeLists.txt @ooooo-create
@Liyulingyue #61947
43 paddle/fluid/distributed/test/CMakeLists.txt @ooooo-create
@Liyulingyue #61948

认领方式

请大家以 comment 的形式认领任务,如:

【报名】:1、3、12-13

多个任务之间需要使用中文顿号分隔,报名多个连续任务可用横线表示,如 2-5
PR 提交格式:在 PR 的标题中以 【paddle_test No.】 开头,注明任务编号

看板信息

任务方向 任务数量 提交作品 / 任务认领 提交率 完成 完成率
快乐开源 43 43 / 43 100.0% 25 58.14%

统计信息

排名不分先后 @Liyulingyue (20) @ccsuzzh (1) @xiaoyewww (4)

@ooooo-create
Copy link
Contributor

【报名】:35-43

@DrRyanHuang
Copy link
Member

【报名】:6-10

@luotao1 luotao1 changed the title [编译优化] paddle_test 推全 【编译优化】paddle_test 推全 Jan 16, 2024
@xiaoyewww
Copy link
Contributor

【报名】:31-34

@luotao1
Copy link
Contributor

luotao1 commented Apr 9, 2024

【编译优化】paddle_test 推全 暂时结项,等后续 cmake 分目录治理工作结束后,再重新启动,感谢参与的小伙伴们!

排名不分先后 @Liyulingyue (20) @ccsuzzh (1) @xiaoyewww (4)

@luotao1 luotao1 closed this as completed Apr 9, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Call for Contributions Apr 9, 2024
@paddle-bot paddle-bot bot added status/close 已关闭 and removed status/new-issue 新建 labels Apr 9, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
HappyOpenSource 快乐开源活动issue与PR status/close 已关闭 type/others 其他问题
Projects
Development

No branches or pull requests

8 participants