Re: Powerpc bootstrap failure due to duplicate case value

2017-01-16 Thread Alan Modra
On Mon, Jan 16, 2017 at 12:41:29PM +0100, Jakub Jelinek wrote: > Or, as a switch it could be of the form: > switch (INSN_CODE (insn)) > { > #ifdef HAVE_ctrsi_internal1 > case CODE_FOR_ctrsi_internal1: > case CODE_FOR_ctrsi_internal2: > case CODE_FOR_ctrsi_internal3: > case COD

Re: Powerpc bootstrap failure due to duplicate case value

2017-01-16 Thread Jakub Jelinek
On Mon, Jan 16, 2017 at 09:53:17PM +1030, Alan Modra wrote: > Commited as obvious. > > PR target/79098 > * config/rs6000/rs6000.c (rs6000_legitimate_combined_insn): Don't > use a switch. Perhaps it would be useful to write why it can't be written as a switch. Or, as a switch it

Powerpc bootstrap failure due to duplicate case value

2017-01-16 Thread Alan Modra
Commited as obvious. PR target/79098 * config/rs6000/rs6000.c (rs6000_legitimate_combined_insn): Don't use a switch. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index 11394b2..f1d5d9d 100644 --- a/gcc/config/rs6000/rs6000.c +++ b/gcc/config/rs6000/