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.

Sign up for GitHub

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

Jump to bottom

Link LLVM tools dynamically #43577

Merged
merged 3 commits into from
Aug 4, 2017
Merged

Link LLVM tools dynamically #43577

merged 3 commits into from
Aug 4, 2017

Commits on Jul 31, 2017

  1. Link LLVM tools dynamically 

    Set `LLVM_LINK_LLVM_DYLIB=ON` -- "If enabled, tools will be linked with
the libLLVM shared library."  Rust doesn't ship any of the LLVM tools,
and only needs a few at all for some test cases, so statically linking
the tools is just a waste of space.  I've also had memory issues on
slower machines with LLVM debuginfo enabled, when several tools start
linking in parallel consuming several GBs each.

With the default configuration, `build/x86_64-unknown-linux-gnu/llvm`
was 1.5GB before, now down to 731MB.  The difference is more drastic
with `--enable-llvm-release-debuginfo`, from 28GB to "only" 13GB.

This does not change the linking behavior of `rustc_llvm`.
    @cuviper
    cuviper committed Jul 31, 2017
    Configuration menu
    Copy the full SHA
    1636a2c View commit details
    Browse the repository at this point in the history

Commits on Aug 3, 2017

  1. Exclude Windows from LLVM_LINK_LLVM_DYLIB

    @cuviper
    cuviper committed Aug 3, 2017
    Configuration menu
    Copy the full SHA
    ced1fda View commit details
    Browse the repository at this point in the history

Commits on Aug 4, 2017

  1. Use LLVM_LINK_LLVM_DYLIB only on linux-gnu and apple-darwin

    @cuviper
    cuviper committed Aug 4, 2017
    Configuration menu
    Copy the full SHA
    6c46f4f View commit details
    Browse the repository at this point in the history