Short-circuit calculate_fallback_to_f32 when no float vars#156139
Short-circuit calculate_fallback_to_f32 when no float vars#156139rust-bors[bot] merged 1 commit intorust-lang:mainfrom
calculate_fallback_to_f32 when no float vars#156139Conversation
|
👍 |
|
@bors r- |
|
This pull request was unapproved. |
|
@bors r+ |
This comment has been minimized.
This comment has been minimized.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 1d72d7e (parent) -> cb40c25 (this PR) Test differencesShow 6 test diffs6 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard cb40c25f6aebb637163d26bf76a680ed6e5d1eda --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (cb40c25): comparison URL. Overall result: ❌✅ regressions and improvements - please read:Our benchmarks found a performance regression caused by this PR. Next Steps:
@rustbot label: +perf-regression 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 0.3%, secondary -0.6%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 3.0%, secondary -18.9%)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: 494.198s -> 497.794s (0.73%) |
|
perf triage:
@rustbot label: +perf-regression-triaged |
Sharing a small fix.
When no unresolved variable is a
float_vid, no f32 fallback can apply,so we can skip
from_float_for_f32_root_vids(which walks the proof treeof every pending obligation under the new solver — O(N × M) on crates
with many large obligations).
On ReShell: stage1 wall-clock 2m35s → 1m46s (-49s, ~31%).
Context + profile breakdown will be linked at
rust-lang/trait-system-refactor-initiative#254 after this PR opens.
r? @lcnr