On 26 November 2015 at 16:55, Matthew Wahab <matthew.wa...@foss.arm.com> wrote:
> Hello,
>
>
> ARMv8.1 includes an extension to ARM which adds two Adv.SIMD
> instructions, vqrdmlah and vqrdmlsh. This patch set adds support for
> ARMv8.1 and for the new instructions, enabling the architecture with
> --march=armv8.1-a. The new instructions are enabled when both ARMv8.1
> and a suitable fpu options are set, for instance with -march=armv8.1-a
> -mfpu=neon-fp-armv8 -mfloat-abi=hard.
>
> This patch set adds the command line options and internal feature
> macros. Following patches
> - enable multilib support for ARMv8.1,
> - add patterns for the new instructions,
> - add the ACLE feature macro for the ARMv8.1 extensions,
> - extend target support in the testsuite to ARMv8.1,
> - add the ACLE intrinsics for vqrmdl{as}h and
> - add the ACLE intrinsics for vqrmdl{as}h_lane.
>
> Tested the series for arm-none-eabi with cross-compiled check-gcc on an
> ARMv8.1 emulator. Also tested arm-none-linux-gnueabihf with native
> bootstrap and make check.
>
> Is this ok for trunk?
> Matthew
>
Hi,

The whole series LGTM, but do you plan to add tests for the new intrinsics?

Thanks,

Christophe.


> gcc/
> 2015-11-26  Matthew Wahab  <matthew.wa...@arm.com>
>
>         * config/arm/arm-arches.def: Add "armv8.1-a" and "armv8.1-a+crc".
>         * config/arm/arm-protos.h (FL2_ARCH8_1): New.
>         (FL2_FOR_ARCH8_1A): New.
>         * config/arm/arm-tables.opt: Regenerate.
>         * config/arm/arm.c (arm_arch8_1): New.
>         (arm_option_override): Set arm_arch8_1.
>         * config/arm/arm.h (TARGET_NEON_RDMA): New.
>         (arm_arch8_1): Declare.
>         * doc/invoke.texi (ARM Options, -march): Add "armv8.1-a" and
>         "armv8.1-a+crc".
>         (ARM Options, -mfpu): Fix a typo.

Reply via email to