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 <ebotca...@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.