Re: [PATCH] PR78056: Fix build failure on Power7

2016-10-26 Thread Segher Boessenkool
On Wed, Oct 26, 2016 at 08:57:11AM -0500, Bill Schmidt wrote: > On Oct 26, 2016, at 8:29 AM, Segher Boessenkool > wrote: > > > > So, you do not want to create the builtins that we expand to machine insns > > that are not supported with the -mcpu= (or other flags) in

Re: [PATCH] PR78056: Fix build failure on Power7

2016-10-26 Thread Bill Schmidt
On Oct 26, 2016, at 8:29 AM, Segher Boessenkool wrote: > > So, you do not want to create the builtins that we expand to machine insns > that are not supported with the -mcpu= (or other flags) in use. What does > the ABI have to say about this? The ABI is silent on

Re: [PATCH] PR78056: Fix build failure on Power7

2016-10-26 Thread Segher Boessenkool
Hi Kelvin, On Wed, Oct 26, 2016 at 02:41:43PM +0200, Segher Boessenkool wrote: > PR target/78056 > * config/rs6000/rs6000.c (spe_init_builtins): Modify loops to not > define builtin functions from the bdesc_spe_predicates or > bdesc_spe_evsel arrays if the builtin mask is

[PATCH] PR78056: Fix build failure on Power7

2016-10-25 Thread Kelvin Nilsen
This patch corrects an error introduced with commit 241314. That patch introduced several new built-in functions to support Power9 string instructions. The error that was found subsequent to the trunk commit is that initialization of the built-in function tables encounters an internal compiler