On Tue, Jul 17, 2018 at 03:44:43PM -0500, Kelvin Nilsen wrote:
> The many PowerPC built-in functions (intrinsics) that are enabled by 
> including <altivec.h> each have different configuration requirements.  To 
> simplify the description of the requirements, this patch sorts these 
> functions into different subsubsections.  
> 
> A subsequent patch will add and remove various functions from each section to 
> correct incompatibilities between what is implemented and what is documented.
> 
> I have built and regression tested this patch on powerpc64le-unknown-linux 
> and on powerpc-linux (P8 big-endian) with no regressions.  I have also built 
> and reviewed the gcc.pdf file.

A changelog nit:

>       * doc/extend.texi (PowerPC AltiVec/VSX Built-in Functions):
>       Corrected spelling of this subsection.  Moved some material to new
>       subsubsections "PowerPC AltiVec Built-in Functions on ISA 2.06" and
>       "PowerPC AltiVec Built-in Functions on ISA 2.07".

        * doc/extend.texi (PowerPC AltiVec Built-in Functions): Rename to...
        (PowerPC AltiVec/VSX Built-in Functions): ... this.  Move some material
        to new subsubsections "PowerPC AltiVec Built-in Functions on ISA 2.06"
        and "PowerPC AltiVec Built-in Functions on ISA 2.07".

(I.e., mention old name as well as new, and use imperative instead of
past tense).

Looks fine otherwise.  Okay for trunk.  Thanks!


Segher

Reply via email to