On Fri, Oct 23, 2015 at 01:22:16PM +0100, Matthew Wahab wrote: > The ARMv8.1 architecture extension adds two Adv.SIMD instructions, > sqrdmlah and sqrdmlsh. This patch adds the feature macro > __ARM_FEATURE_QRDMX to indicate the presence of these instructions, > generating it when the feature is available, as it is when > -march=armv8.1-a is selected. > > Tested the series for aarch64-none-linux-gnu with native bootstrap and > make check on an ARMv8 architecture. Also tested aarch64-none-elf with > cross-compiled check-gcc on an ARMv8.1 emulator. > > Ok for trunk? > Matthew
I don't see this macro documented in the versions of ACLE available from the ARM documentation sites, and googling doesn't show anything other than your patches. You don't explicitly mention anywhere in cover text for this series where these new features are (or will be?) documented. Could you please write a more complete description of where these new macros and intrinsics come from and what they are intended to do? I would not like to accept them without some confidence that these names have been finalized, and I am nervous about having the best description of the behaviour of them be the GCC source code. Richard, Marcus? Thanks, James > > gcc/ > 2015-10-23 Matthew Wahab <matthew.wa...@arm.com> > > * config/aarch64/aarch64-c.c (aarch64_update_cpp_builtins): Add > ARM_FEATURE_QRDMX. >
