Hi Kyrill,

> >
> > Bootstrap and Regtest on aarch64-none-linux-gnu, arm-none-gnueabihf
> > and x86_64-pc-linux-gnu are still on going but previous patch showed no
> regressions.
> >
> 
> Did the testing go okay in the end?

Yes it was clean.

> This looks ok to me but can you provide an example, or better yet, add a test
> that demonstrates this changes?

I've tried, but without the rest of the patch series I can't seem to come with 
an example that enters the given path.
The failure only happens when the vectorizer tries to vectorize to an unknown 
size. 

Most simple math operations would have a known size based on the vectorization 
factor, in any case, I'll hold off this patch then
Until the rest of the series is retried in stage 1.

Thanks,
Tamar

> 
> Thanks,
> Kyrill
> 
> > Ok for trunk?
> >
> > Thanks,
> > Tamar
> >
> > gcc/ChangeLog:
> >
> > 2018-11-11  Tamar Christina  <tamar.christ...@arm.com>
> >
> >         * config/arm/arm.c (arm_preferred_simd_mode): Add V4HF and V8HF.
> >
> > --

Reply via email to