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-llvm.py: Add a note about LLVM commit 7d7771f34d14 for BOLT #189

Merged
merged 1 commit into from May 3, 2022

Conversation

nathanchance
Copy link
Member

LLVM commit 7d7771f34d14 ("[BOLT] Compact legacy profiles") makes the
merged profiles much smaller, which allows less powerful machines to do
BOLT. Add a note about it to the --bolt help text so that users are
aware that performing BOLT without that change will require much more
memory.

Closes: #188
Link: llvm/llvm-project@7d7771f

LLVM commit 7d7771f34d14 ("[BOLT] Compact legacy profiles") makes the
merged profiles much smaller, which allows less powerful machines to do
BOLT. Add a note about it to the '--bolt' help text so that users are
aware that performing BOLT without that change will require much more
memory.

Closes: ClangBuiltLinux#188
Link: llvm/llvm-project@7d7771f
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
@nathanchance
Copy link
Member Author

@torvic9 Please let me know if this help text is adequate for addressing #188 :)

Copy link
Contributor

@torvic9 torvic9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@msfjarvis msfjarvis merged commit 57d3438 into ClangBuiltLinux:main May 3, 2022
@msfjarvis msfjarvis deleted the bolt-note-3 branch May 3, 2022 07:45
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.

[BOLT] Minimum hardware requirements?
3 participants