r? @Jarcho

rustbot has assigned @Jarcho.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

The lint is only wrong because the two breaks have different code before them. These two are the same:

loop {
   let Some(x) = foo() else { bar(); break; };
   baz(x);
}

while let Some(x) = foo() { baz(x); }
bar();

Sure, but hoisting the code before the break out of the loop would be an enhancement to this lint. This PR fixes an oversight in the lint code which causes the reported bug.

Can you change the test to be a case that can't lint and add a note somewhere that hoisting could be done then.

Also thank you for using black_box in the test. That should be done more.

Can you change the test to be a case that can't lint and add a note somewhere that hoisting could be done then.

Done