Skip to content
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

Build GIE ffi wrapper using clang with llvm LTO. #254

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sighingnow
Copy link
Collaborator

@sighingnow sighingnow commented Apr 16, 2021

🤖 Generated by Copilot at fa37867

This pull request enables link-time optimization and uses clang and lld tools for the executor crate and its native runtime library. This improves the performance and reduces the binary size of the interactive_engine component.

Authored-by: Jun Ma liangbin.mj@alibaba-inc.com.

Signed-off-by: Tao He <sighingnow@gmail.com>
@sighingnow sighingnow marked this pull request as draft April 16, 2021 08:40
@codecov-io
Copy link

Codecov Report

Merging #254 (fa37867) into main (1a6b040) will decrease coverage by 12.06%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main     #254       +/-   ##
===========================================
- Coverage   78.70%   66.64%   -12.07%     
===========================================
  Files          57       55        -2     
  Lines        5255     5009      -246     
===========================================
- Hits         4136     3338      -798     
- Misses       1119     1671      +552     
Impacted Files Coverage Δ
python/graphscope/deploy/kubernetes/utils.py 12.43% <0.00%> (-61.14%) ⬇️
python/graphscope/deploy/kubernetes/cluster.py 23.48% <0.00%> (-58.34%) ⬇️
python/graphscope/interactive/query.py 38.46% <0.00%> (-52.02%) ⬇️
...n/graphscope/deploy/kubernetes/resource_builder.py 27.94% <0.00%> (-29.21%) ⬇️
python/graphscope/client/rpc.py 66.12% <0.00%> (-19.06%) ⬇️
python/graphscope/analytical/app/triangles.py 85.71% <0.00%> (-14.29%) ⬇️
python/graphscope/client/session.py 63.00% <0.00%> (-10.52%) ⬇️
python/graphscope/framework/graph.py 84.11% <0.00%> (-8.12%) ⬇️
python/graphscope/framework/loader.py 74.39% <0.00%> (-4.27%) ⬇️
python/graphscope/deploy/launcher.py 72.22% <0.00%> (-3.97%) ⬇️
... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1a6b040...fa37867. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants