On 13 September 2017 at 10:02, Kyrill Tkachov
<kyrylo.tkac...@foss.arm.com> wrote:
>
> Please add a comment here saying that the units are in COSTS_N_INSNS
> so that we can reduce the temptation to use these in inappropriate contexts.
>> + if (VECTOR_MODE_P (mode))
>> + {
>> + *cost += current_tune->addr_mode_costs->vector[op_type];
>> + }
>> + else if (FLOAT_MODE_P (mode))
>> + {
>> + *cost += current_tune->addr_mode_costs->fp[op_type];
>> + }
>> + else
>> + {
>> + *cost += current_tune->addr_mode_costs->integer[op_type];
>> + }
>
>
> No need for brackets for single-statement conditionals.
Done.
