Skip to content

Commit

Permalink
Auto merge of #41967 - ishitatsuyuki:patch-1, r=alexcrichton
Browse files Browse the repository at this point in the history
Reenable opt-level 3

This comment is quite old, let's see what would happen with current MSVC.

Since there's no AppVeyor test for PR, the best way is to try if it get through homu. I don't recommend doing this in roll-up.
  • Loading branch information
bors committed May 20, 2017
2 parents 4662b15 + 30383b2 commit 0bd9e1f
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions src/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,6 @@ members = [
"tools/rls",
]

# 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
[profile.bench]
opt-level = 2

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

0 comments on commit 0bd9e1f

Please sign in to comment.