Skip to content

Commit

Permalink
Revert "Reenable opt-level 3"
Browse files Browse the repository at this point in the history
This reverts commit 30383b2.
  • Loading branch information
Mark-Simulacrum committed May 20, 2017
1 parent a0da1e0 commit 409d40f
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions src/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,14 @@ 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

2 comments on commit 409d40f

@matthiaskrgr
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this still an issue (asking since llvm got upgraded in nightly)?

@Mark-Simulacrum
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy to see another PR trying to remove these.

Please sign in to comment.