<snip>

>
> OK with the comment typos fixed.
>
> > > > gcc/ChangeLog:
> > > >
> > > > 2018-11-11  Tamar Christina  <tamar.christ...@arm.com>
> > > >
> > > >         * config/aarch64/aarch64-simd.md (aarch64_fcadd<rot><mode>,
> > > >         fcadd<rot><mode>3, aarch64_fcmla<rot><mode>,
> > > >         fcmla<rot><mode>4): New.
> > > >         * config/aarch64/aarch64.h (TARGET_COMPLEX): New.
> > > >         * config/aarch64/iterators.md (UNSPEC_FCADD90, UNSPEC_FCADD270,
> > > >         UNSPEC_FCMLA, UNSPEC_FCMLA90, UNSPEC_FCMLA180, 
> > > > UNSPEC_FCMLA270): New.
> > > >         (FCADD, FCMLA): New.
> > > >         (rot, rotsplit1, rotsplit2): New.
> > > >         * config/arm/types.md (neon_fcadd, neon_fcmla): New.
>
> Should we push these to an existing class for now, and split them later when
> someone provides a scheduling model which makes use of them?

Sorry about a blast from the past :P . Perhaps these should just have
been added to is_neon_type that allows for the belts and braces
handling in cortex-a57.md which is the basis for many a scheduler
description in the backend :)

Ramana

Reply via email to