Richard,

Thanks for catching this.
Your change is optimal for X-Gene 1.

—Phil.

> On 15 Jan 2015, at 19:51, Richard Earnshaw <rearn...@arm.com> wrote:
> 
> The recent xgene tuning parameters merge broke the ARM bootstrap, since
> the tables have been extended by an additional parameter giving:
> 
> gcc/config/arm/arm.c:1932:1: error: missing initializer for member
> 'tune_params::fuseable_ops' [-Werror=missing-field-initializers]
> };
> ^
> 
> 
> Fixed as below.  I've no idea if this is the optimial behaviour for
> Xgene parts, but it preserves the previous behaviour of the compiler
> before this pass was added.  I'll let those who really know decide
> whether a more comprehensive change is needed.
> 
> R.
> 
> 2015-01-15  Richard Earnshaw  <rearn...@arm.com>
> 
>       * arm.c (arm_xgene_tune): Add default initializer for instruction
>       fusion.<xgene.patch>

Reply via email to