<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