• Nico Weber's avatar
    Reland "win: Link with lld instead of link.exe by default" · a25e3672
    Nico Weber authored
    The nacl browser_test failures on Win7 that caused the last revert are fixed.
    There's a known bug about this regressing binary size (crbug.com/838449),
    but with in-progress patches binary size will soon be 200 kB smaller and
    these should all be rolled in by branch point, so let's reland this to find
    more unknown unknowns.
    
    Original change's description:
    > win: Link with lld instead of MSVC's link.exe by default
    >
    > lld is LLVM's linker. It produces PE/COFF and PDB files just like
    > link.exe, but it's significantly faster and it can also handle LLVM's
    > internal representation, which will enable us to do link-time
    > optimization and control-flow integraty checks with Clang.
    >
    > While lld is much faster at linking, it doesn't support incremental
    > links, meaning builds that only touch a few files and re-link a large
    > executable may become slower.
    >
    > This is the first attempt at switching everything over, with the
    > purpose of gathering data and finding unknown unknowns. It's likely
    > temporary until something breaks.
    >
    > is_win_fastlink is implicitly ignored when using lld, as lld without
    > fastlink is faster than link.exe with it.
    >
    > Also switch the CrWinClangLLD bots on chromium.clang to use MSVC's
    > link.exe to make sure that configuration keeps working.
    >
    > Bug: 792131
    
    Change-Id: I18aba7a66c54c87092a13745f0ca213171ec25db
    Reviewed-on: https://chromium-review.googlesource.com/1054521
    Commit-Queue: Nico Weber <thakis@chromium.org>
    Reviewed-by: default avatarReid Kleckner <rnk@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#557987}
    a25e3672
mb_config.pyl 73.9 KB