Conversation
This comment has been minimized.
This comment has been minimized.
|
☔ The latest upstream changes (presumably #50016) made this pull request unmergeable. Please resolve the merge conflicts. |
This comment has been minimized.
This comment has been minimized.
|
☔ The latest upstream changes (presumably #50228) made this pull request unmergeable. Please resolve the merge conflicts. |
75d6eed to
66e389d
Compare
|
Here are the changes to Rayon required, which I propose publishing in a |
|
☔ The latest upstream changes (presumably #50290) made this pull request unmergeable. Please resolve the merge conflicts. |
c61e880 to
d7c8e82
Compare
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
michaelwoerister
left a comment
There was a problem hiding this comment.
Looks great, @Zoxc! It's very nice how well this is integrated with Rayon's abstractions.
I hope we can clean up the handling of thread-local state in the future. All this setting and forwarding is becoming a bit unwieldy.
I took a quick look at the changes to Rayon and they look sensible to me but @nikomatsakis still needs to review them before we can merge this.
src/librustc_driver/lib.rs
Outdated
There was a problem hiding this comment.
Maybe Box<TransCrate + sync::Send> could get a type alias like CrateStoreDyn to clean things up.
There was a problem hiding this comment.
I've moved the creation of the thread-pool to an earlier stage, so TransCrate does not require Send.
src/libsyntax_pos/symbol.rs
Outdated
There was a problem hiding this comment.
Did you ever get to the bottom of why this is necessary?
|
Ideally we would merge the rayon changes into upstream if they seem general purpose enough. |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
a8de59c to
f43c046
Compare
@michaelwoerister approved it by IRC
This version of Rayon doesn't use coroutines. |
|
🔒 Merge conflict |
|
@bors r=michaelwoerister |
|
📌 Commit 4afdae6 has been approved by |
Add a Rayon thread pool r? @michaelwoerister
|
☀️ Test successful - status-appveyor, status-travis |
r? @michaelwoerister