On Mon, Jun 06, 2016 at 02:40:45PM +0100, Jiong Wang wrote: > These intrinsics were implemented before "fabd<mode>_3" introduces. > Meanwhile > the patterns "fabd<mode>_3" and "*fabd_scalar<mode>3" can be merged into a > single "fabd<mode>3" using VALLF. > > This patch migrate the implementation to builtins backed by this pattern.
OK, but watch your ChangeLog format and line length. Thanks, James > > gcc/ > 2016-06-01 Jiong Wang <jiong.w...@arm.com> > > * config/aarch64/aarch64-builtins.def (fabd): New builtins > for modes > VALLF. > * config/aarch64/aarch64-simd.md (fabd<mode>_3): Extend > modes from VDQF > to VALLF. Rename to "fabd<mode>3". > "*fabd_scalar<mode>3): Delete. > * config/aarch64/arm_neon.h (vabds_f32): Remove inline assembly. > Use builtin. > (vabdd_f64): Likewise. > (vabd_f32): Likewise. > (vabd_f64): Likewise. > (vabdq_f32): Likewise. > (vabdq_f64): Likewise.