On Fri, May 29, 2026 at 11:17 AM Jeffrey Law <[email protected]> wrote: > > On 5/17/2026 1:03 PM, Andrew Pinski wrote: > > The label for case_bit_test is only used to stability > > the qsort but the target bb here could be used for that. > > So let's remove the label from case_bit_test and stability > > the qsort by using the target bb's index. > > > > Bootstrapped and tested on x86_64-linux-gnu. > > > > gcc/ChangeLog: > > > > * tree-switch-conversion.cc (case_bit_test::cmp): Stability > > based on the index of the target bb instead. > > (bit_test_cluster::emit): Remove setting of the label field > > of case_bit_test. > > * tree-switch-conversion.h (case_bit_test): Remove the label > > field. > > > > Signed-off-by: Andrew Pinski <[email protected]> > Presumably we don't have to worry about blocks with multiple labels > here? Assuming that's the case, then this is OK.
Correct, the main reason it does not need to worry about multiple labels here is because the time the bb is added to this list is when the label is set; it is NOT updated at all. And there is only one entry in this list per BB. Pushed as r17-995-g7c328e9eeadf66f33758764d24151b9eb680c1f0 . Thanks, Andrea > jeff
