On 07/11/14 13:09, Richard Earnshaw wrote:
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.


Thanks. Attached patch committed.

Tejas.

Attachment: intrinsics-doc-4.txt.gz
Description: application/gzip

Reply via email to