On 04/11/14 13:17, Tejas Belagod wrote:
> On 03/11/14 17:58, Joseph Myers wrote:
>> On Mon, 3 Nov 2014, Tejas Belagod wrote:
>>
>>> If I mention in a couple of sentences the level of ACLE support there is in
>>> GCC currently, this section will need to be updated every time there is an
>>> improvement in ACLE support - I guess we'll just have to remember to remove
>>> parts of this section as we do that.
>>
>> Yes, it's generally the case when adding new user-visible features that
>> documentation needs updating. The release notes (gcc-N/changes.html in
>> htdocs) should be updated for any significant new features as well.
>>
>
> Thanks. The AArch64 ACLE CRC32 intrinsics were introduced in 4.9, so its
> not new in 5.0 - https://gcc.gnu.org/gcc-4.9/changes.html. But we've
> improved AArch64 NEON Intrinsics in 5.0 significantly which deserves a
> mention. I'll do that in a separate patch.
>
> Revised patch to fix extend.texi for ACLE attached. OK for trunk?
>
> Thanks,
> Tejas.
>
> 2014-11-04 Tejas Belagod <tejas.bela...@arm.com>
>
> gcc/
> * Makefile.in (TEXI_GCC_FILES): Remove arm-acle-intrinsics.texi,
> arm-neon-intrinsics.texi, aarch64-acle-intrinsics.texi.
> * doc/aarch64-acle-intrinsics.texi: Remove.
> * doc/arm-acle-intrinsics.texi: Remove.
> * doc/arm-neon-intrinsics.texi: Remove.
> * doc/extend.texi: Consolidate sections AArch64 intrinsics,
> ARM NEON Intrinsics, ARM ACLE Intrinsics into one ARM C Language
> Extension section. Add references to public ACLE specification.
>
> +Extensions(ACLE) specification, which can be found at
Space before parenthesis.
> +@node ARM C Language Extensions
I think this should keep the ACLE in parenthesis, so
+@node ARM C Language Extensions (ACLE)
> +back-ends support CRC32 intrinsics from arm_acle.h. ARM backend's 16-bit
@file{} around arm_acle.h. s/ARM/The ARM/
OK with those changes.
R.
