Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Evaluate using Profile-Guided Optimization (PGO) and LLVM BOLT for Perspective C++ data engine #2405

Closed
zamazan4ik opened this issue Oct 29, 2023 · 0 comments

Comments

@zamazan4ik
Copy link

Hi!

Recently I checked Profile-Guided Optimization (PGO) improvements on many projects. All current results are available here. According to the multiple tests, PGO can help with improving performance in many cases (including databases). Since Perspective cares about the performance, I think trying to optimize Perspective's C++ part with PGO would be a good idea.

I can suggest the following action points:

  • Perform PGO benchmarks on Perspective. And if it shows improvements - add a note to the documentation about possible improvements in Perspective performance with PGO.
  • Providing an easier way (e.g. a build option) to build scripts with PGO can be helpful for the end-users and maintainers since they will be able to optimize Perspective according to their own workloads.
  • Optimize pre-built Perspective binaries with PGO.

Since the Perspective native part is the library, I think the Pydantic-core experience can be reused here. Also, Clang supports PGO for shared libraries.

Maybe testing Post-Link Optimization techniques (like LLVM BOLT) would be interesting too (Clang and Rustc already use BOLT as an addition to PGO) but I recommend starting from the usual PGO.

Here are some examples of how PGO optimization is integrated in other projects:

@finos finos locked and limited conversation to collaborators Oct 29, 2023
@texodus texodus converted this issue into discussion #2406 Oct 29, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant