> Marcus Shawcroft wrote: > On 27 April 2015 at 14:43, Wilco Dijkstra <wdijk...@arm.com> wrote: > > >> static unsigned int > >> -aarch64_min_divisions_for_recip_mul (enum machine_mode mode > >> ATTRIBUTE_UNUSED) > >> +aarch64_min_divisions_for_recip_mul (enum machine_mode mode) > >> { > >> - return 2; > >> + if (GET_MODE_UNIT_SIZE (mode) == 4) > >> + return aarch64_tune_params->min_div_recip_mul_sf; > >> + return aarch64_tune_params->min_div_recip_mul_df; > > This should be expressed directly as mode == SFmode (or DFmode) rather > than the indirect approach of first computing the size first.
Can we never see vector types at this point? Wilco