diff options
| author | bors <bors@rust-lang.org> | 2020-10-22 12:24:55 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2020-10-22 12:24:55 +0000 |
| commit | 500ddc5efda9d4b0c8f90b654d220ffc9e24ecb5 (patch) | |
| tree | 1854af90761b22499a97c27be2d3100aedfa97df /compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp | |
| parent | 6b9fbf212a06944ff23325d2d63215805dc3c6c3 (diff) | |
| parent | 52cedcab9221bd63a67f4de4cd9c577518ed0d3b (diff) | |
| download | rust-500ddc5efda9d4b0c8f90b654d220ffc9e24ecb5.tar.gz rust-500ddc5efda9d4b0c8f90b654d220ffc9e24ecb5.zip | |
Auto merge of #77871 - Julian-Wollersberger:less-query-context, r=oli-obk
Make fewer types generic over QueryContext While trying to refactor `rustc_query_system::query::QueryContext` to make it dyn-safe, I noticed some smaller things: * QueryConfig doesn't need to be generic over QueryContext * ~~The `kind` field on QueryJobId is unused~~ * Some unnecessary where clauses * Many types in `job.rs` where generic over `QueryContext` but only needed `QueryContext::Query`. If handle_cycle_error() could be refactored to not take `error: CycleError<CTX::Query>`, all those bounds could be removed as well. Changing `find_cycle_in_stack()` in job.rs to not take a `tcx` argument is the only functional change here. Everything else is just updating type signatures. (aka compile-error driven development ^^) ~~Currently there is a weird bug where memory usage suddenly skyrockets when running UI tests. I'll investigate that tomorrow. A perf run probably won't make sense before that is fixed.~~ EDIT: `kind` actually is used by `Eq`, and re-adding it fixed the memory issue.
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp')
0 files changed, 0 insertions, 0 deletions
