Rerun the goal with doubled recursion limit even if the goal has ty vars - #160254
Conversation
|
@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.
[PERF] rerun even if the goal has ty vars
This comment has been minimized.
This comment has been minimized.
|
Finished benchmarking commit (9c22651): comparison URL. Overall result: no relevant changes - no action neededBenchmarking means the PR may be perf-sensitive. Consider adding rollup=never if this change is not fit for rolling up. @rustbot label: -S-waiting-on-perf -perf-regression Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)Results (primary -1.9%, secondary 5.8%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -0.3%, secondary -0.4%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary -0.1%, secondary -0.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 492.09s -> 490.546s (-0.31%) |
|
alright, can you make sure this fixes the issues and add them as test, then r=me I guess 🤣 |
8066f2e to
1489e47
Compare
|
r? @mu001999 rustbot has assigned @mu001999. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
Sorry, forgot to |
Rollup merge of #160254 - adwinwhite:fix-fcw-missing, r=lcnr Rerun the goal with doubled recursion limit even if the goal has ty vars Fix the `bevy_render` regression and more in the next solver crater run. The problem is that we don't rerun the root goal with doubled recursion limit if it contains ty vars. But we actually need the evaluation response to resolve the ty vars. So the type inference fails.
Fix the
bevy_renderregression and more in the next solver crater run.The problem is that we don't rerun the root goal with doubled recursion limit if it contains ty vars.
But we actually need the evaluation response to resolve the ty vars. So the type inference fails.