Use a single lifetime for MIR construction#61426
Conversation
rust-highfive
added
the
S-waiting-on-review
There was a problem hiding this comment.
You can replace headers like these with impl Builder<'_, 'tcx> if you want.
There was a problem hiding this comment.
I feel like this is a job for rustfmt, not me =P
There was a problem hiding this comment.
Not rustfmt but maybe rustfix. Don't know if we have a lint for it yet though.
src/librustc_mir/build/mod.rs
Outdated
There was a problem hiding this comment.
This can be TyCtxt<'_, 'tcx, 'tcx> without the <'a, 'tcx> on the previous line.
eddyb
left a comment
eddyb
left a comment
There was a problem hiding this comment.
LGTM, modulo potential further lifetime cleanups. cc @pnkfelix @nikomatsakis @matthewjasper
|
☔ The latest upstream changes (presumably #61276) made this pull request unmergeable. Please resolve the merge conflicts. |
Zoxc
force-pushed
the
just-tcx-mir-building
branch
from
1d138de to
b23f682
Compare
|
@bors try |
|
⌛ Trying commit b23f682e98354be01ebee7081e50b2110a7228fb with merge c0d6d4a069cb719fc05ca0a0aa4700f4e4a16600... |
|
☀️ Try build successful - checks-travis |
|
@rust-timer build c0d6d4a069cb719fc05ca0a0aa4700f4e4a16600 |
|
Success: Queued c0d6d4a069cb719fc05ca0a0aa4700f4e4a16600 with parent 7096ff0, comparison URL. |
|
Finished benchmarking try commit c0d6d4a069cb719fc05ca0a0aa4700f4e4a16600, comparison URL. |
Zoxc
force-pushed
the
just-tcx-mir-building
branch
from
b23f682 to
5d8290c
Compare
Zoxc
force-pushed
the
just-tcx-mir-building
branch
from
5d8290c to
d3e1181
Compare
|
@bors r+ rollup=never p=5 |
|
📌 Commit d3e1181 has been approved by |
bors
added
S-waiting-on-bors
|
☀️ Test successful - checks-travis, status-appveyor |
5 participants
Builds on #57214
r? @eddyb