Skip to content

Commit

Permalink
Rollup merge of rust-lang#48204 - matthiaskrgr:RUSTFLAGS_O2_to_O3, r=…
Browse files Browse the repository at this point in the history
…kennytm

change opt-level 2 to 3 in bootstrap rustflags

Since llvm got updated, maybe this also resolved unexpected crashes.
See discussion here: rust-lang@409d40f

Is it possible to give this a test-run?
//cc @Mark-Simulacrum
  • Loading branch information
Manishearth committed Feb 25, 2018
2 parents c7564a7 + 07033a4 commit 8b8a2fb
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions src/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -40,13 +40,10 @@ members = [
"tools/rls/test_data/workspace_symbol",
]

# Curiously, compiletest will segfault if compiled with opt-level=3 on 64-bit
# MSVC when running the compile-fail test suite when a should-fail test panics.
# But hey if this is removed and it gets past the bots, sounds good to me.
[profile.release]
opt-level = 2
opt-level = 3
[profile.bench]
opt-level = 2
opt-level = 3

# These options are controlled from our rustc wrapper script, so turn them off
# here and have them controlled elsewhere.
Expand Down

0 comments on commit 8b8a2fb

Please sign in to comment.