rustc: Allow target-specific default cgus #45032
Merged
+56
−50
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
r? @pnkfelix (rust_highfive has picked a reviewer for you, use r? to override) |
aidanhs
added
the
S-waiting-on-review
|
☔ The latest upstream changes (presumably #45035) made this pull request unmergeable. Please resolve the merge conflicts. |
alexcrichton
force-pushed
the
target-cfu
branch
from
9b7bafb to
77a4037
Compare
|
☔ The latest upstream changes (presumably #45064) made this pull request unmergeable. Please resolve the merge conflicts. |
|
r=me with conflicts resolved. |
Some targets, like msp430 and nvptx, don't work with multiple codegen units right now for bugs or fundamental reasons. To expose this allow targets to express a default. Closes rust-lang#45000
alexcrichton
force-pushed
the
target-cfu
branch
from
77a4037 to
5187763
Compare
|
@bors: r=michaelwoerister |
|
📌 Commit 5187763 has been approved by |
|
⌛ Testing commit 5187763 with merge a5508d30e83ed5f6f9431d3c5590b7339e3c8d11... |
|
💔 Test failed - status-travis |
|
@bors retry
|
bors
added a commit
that referenced
this pull request
Oct 13, 2017
rustc: Allow target-specific default cgus Some targets, like msp430 and nvptx, don't work with multiple codegen units right now for bugs or fundamental reasons. To expose this allow targets to express a default. Closes #45000
kennytm
added
S-waiting-on-bors
|
☀️ Test successful - status-appveyor, status-travis |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some targets, like msp430 and nvptx, don't work with multiple codegen units
right now for bugs or fundamental reasons. To expose this allow targets to
express a default.
Closes #45000