-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
【编译优化】paddle_test 推全 #60793
Labels
Comments
Closed
This was referenced Jan 15, 2024
【报名】:35-43 |
【报名】:6-10 |
【报名】:31-34 |
This was referenced Jan 17, 2024
This was referenced Jan 18, 2024
This was referenced Jan 22, 2024
Open
This was referenced Feb 6, 2024
This was referenced Feb 21, 2024
【编译优化】paddle_test 推全 暂时结项,等后续 cmake 分目录治理工作结束后,再重新启动,感谢参与的小伙伴们!
|
github-project-automation
bot
moved this from In Progress
to Done
in Call for Contributions
Apr 9, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
问题描述 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,好处在于:
二、存在的问题
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
四、任务划分
@Liyulingyue #61985
@Liyulingyue #61984
@Liyulingyue #61980
@Liyulingyue #61981
@Liyulingyue #61983
@Liyulingyue #62031
@Liyulingyue #62034 #62033
@Liyulingyue #62036 #62033
@Liyulingyue #62035 #62033
@Liyulingyue #61943
@Liyulingyue #61944
@Liyulingyue #61945
@Liyulingyue #61946
@Liyulingyue #61947
@Liyulingyue #61948
认领方式
请大家以 comment 的形式认领任务,如:
多个任务之间需要使用中文顿号分隔,报名多个连续任务可用横线表示,如 2-5
PR 提交格式:在 PR 的标题中以 【paddle_test No.】 开头,注明任务编号
看板信息
统计信息
The text was updated successfully, but these errors were encountered: