Skip to content

Commit

Permalink
Auto merge of #66625 - michaelwoerister:test-thinlto-instr-import-lim…
Browse files Browse the repository at this point in the history
…it, r=<try>

Test setting a lower import limit for ThinLTO.

The Firefox build system [sets a lower ThinLTO import limit](https://searchfox.org/mozilla-central/search?q=-import-instr-limit&path=) since recently but because they discovered that that can significantly lower build times without really affecting runtime performance too much. Let's see how this would affect our benchmarks.

r? @ghost
  • Loading branch information
bors committed Nov 26, 2019
2 parents 797fd92 + 18dcb29 commit a2cd298
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/librustc_codegen_llvm/llvm_util.rs
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,8 @@ unsafe fn configure_llvm(sess: &Session) {
}
}

add("-import-instr-limit=75");

if sess.target.target.target_os == "emscripten" &&
sess.panic_strategy() == PanicStrategy::Unwind {
add("-enable-emscripten-cxx-exceptions");
Expand Down

0 comments on commit a2cd298

Please sign in to comment.