Introduce rustc_interface and move some methods there#58250
Introduce rustc_interface and move some methods there#58250bors merged 1 commit intorust-lang:masterfrom
Conversation
a2f59bf to
72ba6f8
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 |
9d8d8d1 to
772325d
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 |
772325d to
a7f0143
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 |
a7f0143 to
fc02bd0
Compare
src/librustc_interface/util.rs
Outdated
There was a problem hiding this comment.
Should this be a doc comment?
|
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 |
fc02bd0 to
0fafff9
Compare
src/librustc/ty/context.rs
Outdated
There was a problem hiding this comment.
And/or why was this there at all?
There was a problem hiding this comment.
It was there after we modified GlobalCtxt to be Sync, but didn't yet actually share it across threads.
src/librustc_driver/lib.rs
Outdated
There was a problem hiding this comment.
This should be a method on Input
There was a problem hiding this comment.
A doc comment would be neat, too
src/librustc_interface/passes.rs
Outdated
There was a problem hiding this comment.
| // Returns all the paths that correspond to generated files. | |
| /// Returns all the paths that correspond to generated files. |
|
☔ The latest upstream changes (presumably #58254) made this pull request unmergeable. Please resolve the merge conflicts. |
src/librustc_driver/driver.rs
Outdated
There was a problem hiding this comment.
This should return the error, shouldnt't it.
There was a problem hiding this comment.
We pass the compile status to the closure and expect it to handle it. I'll change phase_3_run_analysis_passes to return just R.
0fafff9 to
024f7d5
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 |
d7d0cec to
9e4ec00
Compare
|
@bors r=oli-obk |
|
📌 Commit 9e4ec00dba073f668fe777eb6678dd612d3c2770 has been approved by |
|
☔ The latest upstream changes (presumably #58389) made this pull request unmergeable. Please resolve the merge conflicts. |
9e4ec00 to
356da36
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 |
356da36 to
0e4fa4f
Compare
|
☔ The latest upstream changes (presumably #58341) made this pull request unmergeable. Please resolve the merge conflicts. |
|
📌 Commit 6d573d8f971f48d21843cd4407662f14323f74d3 has been approved by |
|
🔒 Merge conflict This pull request and the master branch diverged in a way that cannot be automatically merged. Please rebase on top of the latest master branch, and let the reviewer approve again. How do I rebase?Assuming
You may also read Git Rebasing to Resolve Conflicts by Drew Blessing for a short tutorial. Please avoid the "Resolve conflicts" button on GitHub. It uses Sometimes step 4 will complete without asking for resolution. This is usually due to difference between how Error message |
|
☔ The latest upstream changes (presumably #57760) made this pull request unmergeable. Please resolve the merge conflicts. |
6d573d8 to
23a51f9
Compare
|
@bors r=oli-obk |
|
📌 Commit 23a51f9 has been approved by |
|
☀️ Test successful - checks-travis, status-appveyor |
…ister Refactor passes and pass execution to be more parallel For `syntex_syntax` (with 16 threads and 8 cores): - Cuts `misc checking 1` from `0.096s` to `0.08325s`. - Cuts `misc checking 2` from `0.3575s` to `0.2545s`. - Cuts `misc checking 3` from `0.34625s` to `0.21375s`. - Cuts `wf checking` from `0.3085s` to `0.05025s`. Reduces overall execution time for `syntex_syntax` (with 8 threads and cores) from `4.92s` to `4.34s`. Subsumes rust-lang#58494 Blocked on rust-lang#58250 r? @michaelwoerister
…ister Refactor passes and pass execution to be more parallel For `syntex_syntax` (with 16 threads and 8 cores): - Cuts `misc checking 1` from `0.096s` to `0.08325s`. - Cuts `misc checking 2` from `0.3575s` to `0.2545s`. - Cuts `misc checking 3` from `0.34625s` to `0.21375s`. - Cuts `wf checking` from `0.3085s` to `0.05025s`. Reduces overall execution time for `syntex_syntax` (with 8 threads and cores) from `4.92s` to `4.34s`. Subsumes rust-lang#58494 Blocked on rust-lang#58250 r? @michaelwoerister
…ister Refactor passes and pass execution to be more parallel For `syntex_syntax` (with 16 threads and 8 cores): - Cuts `misc checking 1` from `0.096s` to `0.08325s`. - Cuts `misc checking 2` from `0.3575s` to `0.2545s`. - Cuts `misc checking 3` from `0.34625s` to `0.21375s`. - Cuts `wf checking` from `0.3085s` to `0.05025s`. Reduces overall execution time for `syntex_syntax` (with 8 threads and cores) from `4.92s` to `4.34s`. Subsumes rust-lang#58494 Blocked on rust-lang#58250 r? @michaelwoerister
Split out from #56732
r? @oli-obk