This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [rs6000] Fix 32-bit bootstrap failure
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Eric Botcazou <ebotcazou at adacore dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sun, 1 May 2016 01:48:03 -0500
- Subject: Re: [rs6000] Fix 32-bit bootstrap failure
- Authentication-results: sourceware.org; auth=none
- References: <4992006 dot Ijr1NlHkR2 at polaris>
On Sat, Apr 30, 2016 at 11:51:15AM +0200, Eric Botcazou wrote:
> it's the recurring issue documented in rs6000_expand_ternop_builtin:
>
> Note that a switch statement instead of the sequence of tests
> would be incorrect as many of the CODE_FOR values could be
> CODE_FOR_nothing and that would yield multiple alternatives
> with identical values.
>
> Bootstrapped on PowerPC/Linux, OK for the mainline?
Thanks for finding and fixing it! Please apply.
Segher
> 2016-04-30 Eric Botcazou <ebotcazou@adacore.com>
>
> * config/rs6000/rs6000.c (altivec_expand_lv_builtin): Do not use
> switch statement on instruction code. Remove trailing spaces.
> (altivec_expand_stv_builtin): Likewise.