about summary refs log tree commit diff
path: root/compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2020-10-22 12:24:55 +0000
committerbors <bors@rust-lang.org>2020-10-22 12:24:55 +0000
commit500ddc5efda9d4b0c8f90b654d220ffc9e24ecb5 (patch)
tree1854af90761b22499a97c27be2d3100aedfa97df /compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp
parent6b9fbf212a06944ff23325d2d63215805dc3c6c3 (diff)
parent52cedcab9221bd63a67f4de4cd9c577518ed0d3b (diff)
downloadrust-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