This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][ARM] Remove unused vtrn, vzip, vuzp NEON builtins


On 05/06/14 14:37, Kyrill Tkachov wrote:
Hi all,

This patch removes the NEON builtin functions for vtrn, vzip, vuzp and their
associated wiring and machine descriptions. The builtins were initially used to
implement the corresponding intrinsics in arm_neon.h but those have since been
reimplemented using the more generic __builtin_shuffle. This allows us to clean
up the NEON builtin initialisation code somewhat.

Tested arm-none-eabi on qemu. Made sure that the recently added tests in
simd.exp for vzip, vuzp etc. pass.

Ok for trunk?

Yes this is OK and thanks for the cleanup.

Sorry about the late review.

Ramana


Thanks,
Kyrill

2014-05-06  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

      * config/arm/arm.c (neon_itype): Remove NEON_RESULTPAIR.
      (arm_init_neon_builtins): Remove handling of NEON_RESULTPAIR.
      Remove associated type declarations and initialisations.
      (arm_expand_neon_builtin): Likewise.
      (neon_emit_pair_result_insn): Delete.
      * config/arm/arm_neon_builtins (vtrn, vzip, vuzp): Delete.
      * config/arm/neon.md (neon_vtrn<mode>): Delete.
      (neon_vzip<mode>): Likewise.
      (neon_vuzp<mode>): Likewise.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]