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.


Reply via email to