Skip to content

Commit

Permalink
Rollup merge of rust-lang#41967 - ishitatsuyuki:patch-1, r=petrochenkov
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
frewsxcv committed May 13, 2017
2 parents 8755896 + 30383b2 commit 30a6e38
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 @@ -21,14 +21,6 @@ exclude = [
"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 30a6e38

Please sign in to comment.