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.
jeff
