Skip to content

Commit

Permalink
Rollup merge of #121374 - Nadrieril:factor-explain, r=matthewjasper
Browse files Browse the repository at this point in the history
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/rust#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 73937dd + 33acda0 commit 0fe7932
Showing 0 changed files with 0 additions and 0 deletions.

0 comments on commit 0fe7932

Please sign in to comment.