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
make all cpp tests dynamic linked to libpaddle.so [except windows] #47088
make all cpp tests dynamic linked to libpaddle.so [except windows] #47088
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
da6752b
to
3254dfd
Compare
14be5aa
to
b087c30
Compare
b087c30
to
7006947
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for CMakeLists.txt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…addlePaddle#47088) * make all cpp tests dynamic linked to libpaddle.so * add comments * keep old cc_test for some tests * fix some ut * make some ut use cc_test_old * fix typos and fit for win32 * fix lib path * fix some tests * skip lite test * fit for rocm * fit for cinn * fit for mac * fit for win32 * skip inference ut * skip windows * fix coverage
PR types
Others
PR changes
Others
Describe
make all cpp tests dynamic linked to libpaddle.so
see the comments added in
paddle/CMakeLists.txt
for detailsWhy
libpaddle.so
providedResult
The build directory of coverage CI decreased from 168G to 117G, 51G reduced, 30%.
PS0. There are 229 tests using dynamic link now and still some cpp tests use static link, configured using
cc_test_old
inCMakeLists.txt
.PS1. The dynamic library on windows encountered
unresolved symbol
problem, so it is not supported now.PS2. Found some tests contains symbols that is not in
libpaddle.so
, it means that the code is unused. So here are some PRs to clean the related code.#47103