rs6000: Fix bootstrap
authorJakub Jelinek <jakub@redhat.com>
Wed, 19 Jan 2022 11:22:02 +0000 (12:22 +0100)
committerJakub Jelinek <jakub@redhat.com>
Wed, 19 Jan 2022 11:22:02 +0000 (12:22 +0100)
commit7f3915b83ed37516a2ff84eb3a6a6e37d3fc9a15
treeb7ad5948fea5bf6a0fc36c155de785ace7e81277
parent34ba4275dcf5162efb9b634e4665734766faf4e3
rs6000: Fix bootstrap

> On 18/01/2022 22:42, Segher Boessenkool wrote:
> > > +    default:
> > > +      break;
> > Please don't do that.  You can do
> >
> >    default:
> >      break;
> >      break;
> >      /* And just to make sure:  */
> >      break;
> >      break;
> >
> > and it will do exactly the same as not having a default at all.  Not
> > having such useless code is by far the most readable, so please don't
> > include a default case at all.
>
> I removed the default case. I hope this is what you wanted.

Unfortunately the removal of default: break; breaks bootstrap:
../../gcc/config/rs6000/rs6000.cc: In function ‘const char* rs6000_machine_from_flags()’:
../../gcc/config/rs6000/rs6000.cc:5940:10: error: enumeration value ‘PROCESSOR_PPC601’ not handled in switch [-Werror=switch]
 5940 |   switch (rs6000_cpu)
      |          ^
../../gcc/config/rs6000/rs6000.cc:5940:10: error: enumeration value ‘PROCESSOR_PPC603’ not handled in switch [-Werror=switch]
...
default: break; is needed to tell the -Wswitch warning that it is intentional
that not all enumerators are handled in the switch.

2022-01-19  Jakub Jelinek  <jakub@redhat.com>

* config/rs6000/rs6000.cc (rs6000_machine_from_flags): Add default:.
gcc/config/rs6000/rs6000.cc