| CARVIEW |
Select Language
HTTP/2 200
date: Sun, 27 Jul 2025 19:29:06 GMT
content-type: text/html; charset=utf-8
cache-control: no-cache
content-security-policy: default-src 'none'; base-uri 'self'; child-src github.githubassets.com github.com/assets-cdn/worker/ github.com/assets/ gist.github.com/assets-cdn/worker/; connect-src 'self' uploads.github.com www.githubstatus.com collector.github.com raw.githubusercontent.com api.github.com github-cloud.s3.amazonaws.com github-production-repository-file-5c1aeb.s3.amazonaws.com github-production-upload-manifest-file-7fdce7.s3.amazonaws.com github-production-user-asset-6210df.s3.amazonaws.com *.rel.tunnels.api.visualstudio.com wss://*.rel.tunnels.api.visualstudio.com objects-origin.githubusercontent.com copilot-proxy.githubusercontent.com proxy.individual.githubcopilot.com proxy.business.githubcopilot.com proxy.enterprise.githubcopilot.com *.actions.githubusercontent.com wss://*.actions.githubusercontent.com productionresultssa0.blob.core.windows.net/ productionresultssa1.blob.core.windows.net/ productionresultssa2.blob.core.windows.net/ productionresultssa3.blob.core.windows.net/ productionresultssa4.blob.core.windows.net/ productionresultssa5.blob.core.windows.net/ productionresultssa6.blob.core.windows.net/ productionresultssa7.blob.core.windows.net/ productionresultssa8.blob.core.windows.net/ productionresultssa9.blob.core.windows.net/ productionresultssa10.blob.core.windows.net/ productionresultssa11.blob.core.windows.net/ productionresultssa12.blob.core.windows.net/ productionresultssa13.blob.core.windows.net/ productionresultssa14.blob.core.windows.net/ productionresultssa15.blob.core.windows.net/ productionresultssa16.blob.core.windows.net/ productionresultssa17.blob.core.windows.net/ productionresultssa18.blob.core.windows.net/ productionresultssa19.blob.core.windows.net/ github-production-repository-image-32fea6.s3.amazonaws.com github-production-release-asset-2e65be.s3.amazonaws.com insights.github.com wss://alive.github.com api.githubcopilot.com api.individual.githubcopilot.com api.business.githubcopilot.com api.enterprise.githubcopilot.com; font-src github.githubassets.com; form-action 'self' github.com gist.github.com copilot-workspace.githubnext.com objects-origin.githubusercontent.com; frame-ancestors 'none'; frame-src viewscreen.githubusercontent.com notebooks.githubusercontent.com; img-src 'self' data: blob: github.githubassets.com media.githubusercontent.com camo.githubusercontent.com identicons.github.com avatars.githubusercontent.com private-avatars.githubusercontent.com github-cloud.s3.amazonaws.com objects.githubusercontent.com release-assets.githubusercontent.com secured-user-images.githubusercontent.com/ user-images.githubusercontent.com/ private-user-images.githubusercontent.com opengraph.githubassets.com copilotprodattachments.blob.core.windows.net/github-production-copilot-attachments/ github-production-user-asset-6210df.s3.amazonaws.com customer-stories-feed.github.com spotlights-feed.github.com objects-origin.githubusercontent.com *.githubusercontent.com; manifest-src 'self'; media-src github.com user-images.githubusercontent.com/ secured-user-images.githubusercontent.com/ private-user-images.githubusercontent.com github-production-user-asset-6210df.s3.amazonaws.com gist.github.com; script-src github.githubassets.com; style-src 'unsafe-inline' github.githubassets.com; upgrade-insecure-requests; worker-src github.githubassets.com github.com/assets-cdn/worker/ github.com/assets/ gist.github.com/assets-cdn/worker/
referrer-policy: no-referrer-when-downgrade
server-timing: pull_request_layout-fragment;desc="pull_request_layout fragment";dur=249.700158,conversation_content-fragment;desc="conversation_content fragment";dur=1004.604127,conversation_sidebar-fragment;desc="conversation_sidebar fragment";dur=340.345239,nginx;desc="NGINX";dur=0.45768,glb;desc="GLB";dur=100.914746
strict-transport-security: max-age=31536000; includeSubdomains; preload
vary: X-PJAX, X-PJAX-Container, Turbo-Visit, Turbo-Frame, X-Requested-With,Accept-Encoding, Accept, X-Requested-With
x-content-type-options: nosniff
x-frame-options: deny
x-voltron-version: a2eb102
x-xss-protection: 0
server: github.com
content-encoding: gzip
accept-ranges: bytes
set-cookie: _gh_sess=BNlBN0TOvTeJfwufokj85UGDK%2F%2BoyogRyaPU87ZnSn1%2BhRallBO757qcKC%2BDb%2F7xrWiVvh5wj6RMqm5rouoPFYWPzjxK98XUeM3rAs31FYNkxgAt0kTNiNjdER93Ed3okEWrAHHCxPTFp8p3qT6ZJE7GpxW%2BpIxU%2FyFiZXACKSskk2LaY5HsQtzRt5w282igeIMwPeirbi5F2fP1TOgGZepq8dS2BlrkonJ6k0vcjnvdui7hPFzVGrIra3zFOy6vLb2RvPH62TkICKzOdJluwQ%3D%3D--8P54PemQc5icuUgm--oeEiR5SmP6LYb%2BI6yvm%2FeA%3D%3D; Path=/; HttpOnly; Secure; SameSite=Lax
set-cookie: _octo=GH1.1.1586516529.1753644545; Path=/; Domain=github.com; Expires=Mon, 27 Jul 2026 19:29:05 GMT; Secure; SameSite=Lax
set-cookie: logged_in=no; Path=/; Domain=github.com; Expires=Mon, 27 Jul 2026 19:29:05 GMT; HttpOnly; Secure; SameSite=Lax
x-github-request-id: 9758:0A61:79ADE8:9E458E:68867E01
Link cpp test to libpaddle.so/libpaddle.dll by winter-wang · Pull Request #56691 · PaddlePaddle/Paddle · GitHub
winter-wang
force-pushed
the
fix_compile_kill_9_bug
branch
29 times, most recently
from
August 28, 2023 08:52
winter-wang
force-pushed
the
fix_compile_kill_9_bug
branch
5 times, most recently
from
September 25, 2023 08:08
winter-wang
force-pushed
the
fix_compile_kill_9_bug
branch
from
September 25, 2023 12:37
risemeup1
changed the title
Fix compile kill 9 bug
Link cpp test to libpaddle.so/libpaddle.dll
Sep 26, 2023
Skip to content
Navigation Menu
{{ message }}
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Link cpp test to libpaddle.so/libpaddle.dll #56691
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
Merged
winter-wang
merged 1 commit into
PaddlePaddle:develop
from
winter-wang:fix_compile_kill_9_bug
Sep 26, 2023
Merged
Link cpp test to libpaddle.so/libpaddle.dll #56691
winter-wang
merged 1 commit into
PaddlePaddle:develop
from
winter-wang:fix_compile_kill_9_bug
Sep 26, 2023
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
你的PR提交成功，感谢你对开源项目的贡献! |
winter-wang
force-pushed
the
fix_compile_kill_9_bug
branch
29 times, most recently
from
50b5b08 to
50faf51
Compare
winter-wang
force-pushed
the
fix_compile_kill_9_bug
branch
5 times, most recently
from
9d01e92 to
dbe9f55
Compare
risemeup1
previously approved these changes
Sep 25, 2023
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
winter-wang
force-pushed
the
fix_compile_kill_9_bug
branch
from
dbe9f55 to
bea6d0a
Compare
risemeup1
approved these changes
Sep 26, 2023
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
risemeup1
changed the title
YuanRisheng
approved these changes
Sep 26, 2023
YuanRisheng
reviewed
Sep 26, 2023
JiabinYang
approved these changes
Sep 26, 2023
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.
This file can be deleted
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.
全局搜索了一下egr::scale符号，这个函数在很多单测文件中进行了大量使用(benchmark_utils.cc、forward_autograd_test.cc、fwd_bwd_joint_test.cc)。直接删除该文件的话，相应的单测也需要修改，不然会编译失败。因此这个文件可能需要动态图相关的同学来帮忙进行删除。
zhwesky2010
approved these changes
Sep 26, 2023
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
zhiqiu
approved these changes
Sep 26, 2023
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
zhangbo9674
approved these changes
Sep 26, 2023
Aurelius84
approved these changes
Sep 26, 2023
XiaoguangHu01
approved these changes
Sep 26, 2023
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
chenwhql
approved these changes
Sep 26, 2023
This was referenced Sep 26, 2023
Frida-a
pushed a commit
to Frida-a/Paddle
that referenced
this pull request
Oct 14, 2023
jiahy0825
pushed a commit
to jiahy0825/Paddle
that referenced
this pull request
Oct 16, 2023
danleifeng
pushed a commit
to danleifeng/Paddle
that referenced
this pull request
Nov 14, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
None yet
10 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
You can’t perform that action at this time.
PR types
Bug fixes
PR changes
Others
Description
修复CI流水线中，单测体积过大问题。
目前的很多单测，对Paddle框架通过静态库(.a、.lib)的形式进行链接，导致单个单测的体积过大（超过1G）。在CI流水线编译中，当并发线程过多，就容易因为内存空间不足而使编译失败。
本pr内容如下：
将部分单测修改为对Paddle框架通过动态库(.so、.dll)的形式进行链接。多个单测共享同一份动态库，有效减少内存的占用。
提供了TEST_API宏定义， 在定义了WITH_TESTING的情况下，TEST_API修饰的对象会被编译为动态库的导出符号，方便单测进行使用。
提供了 paddle_test函数， 该函数会自动链接libpaddle.so或者libpaddle.dll动态库。当单个单测体积过大时，建议通过paddle_test链接动态库进行单元测试。
本pr合入前， coverage单测编译后，build目录的大小为 *G, 合入后， build目录的大小为**G。
本pr主要是打通了windows下和linux下同时使用动态库进行测试的道路。后续会逐步将所有大于**M的单测，都改为通过动态库进行链接测试。
Todo:
@risemeup1
将现有的cc_test函数的使用都替换为paddle_test, 移除cc_test函数。
作为对paddle_test函数功能的补充：
当单个单测的大小超过**M的时候，通过CI进行拦截。
Other
Pcard-67164