Skip to content

Commit

Permalink
Rollup merge of rust-lang#121374 - Nadrieril:factor-explain, r=matthe…
Browse files Browse the repository at this point in the history
…wjasper

match lowering: Split off `test_candidates` into several functions and improve comments

The logic of `test_candidates` has three steps: pick a test, sort the candidates, and generate code for everything. So I split it off into three methods.

I also ended up reworking the comments that explain the algorithm. In particular I added detailed examples. I removed the digression about rust-lang#29740 because it's no longer relevant to how the code is structured today.

r? ``@matthewjasper``
  • Loading branch information
matthiaskrgr committed Feb 24, 2024
2 parents ed75229 + de4efa5 commit f4ba47f
Showing 1 changed file with 218 additions and 151 deletions.
Loading

0 comments on commit f4ba47f

Please sign in to comment.