Remove INCR const param from try_execute_query.#155282
Remove INCR const param from try_execute_query.#155282nnethercote wants to merge 1 commit intorust-lang:mainfrom
INCR const param from try_execute_query.#155282Conversation
`try_execute_query` has non-incremental and incremental paths. Currently the non-incr/incr distinction is encoded in two different ways: the `INCR` const bool param and the `dep_node` option value. We don't need both -- `true` is always paired with `Some` and `false` is always paired with `None`. This commit removes `INCR`.
|
@bors try @rust-timer queue |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Remove `INCR` const param from `try_execute_query`.
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (2d480f7): comparison URL. Overall result: ❌ regressions - please read:Benchmarking means the PR may be perf-sensitive. It's automatically marked not fit for rolling up. Overriding is possible but disadvised: it risks changing compiler perf. Next, please: If you can, justify the regressions found in this try perf run in writing along with @bors rollup=never Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (primary -4.8%, secondary -2.5%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 7.3%, secondary -0.7%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 489.054s -> 487.828s (-0.25%) |
try_execute_queryhas non-incremental and incremental paths. Currently the non-incr/incr distinction is encoded in two different ways: theINCRconst bool param and thedep_nodeoption value. We don't need both --trueis always paired withSomeandfalseis always paired withNone. This commit removesINCR.