Hi!

On Wed, May 09, 2018 at 12:42:11PM -0500, Kelvin Nilsen wrote:
> 1. Change the name of the first PowerPC built-in section from 
>    "PowerPC Built-in Functions" to "Basic PowerPC Built-in Functions".
>    This section has never described all PowerPC built-in functions.

After the patch it looks like this:

* NDS32 Built-in Functions::
* picoChip Built-in Functions::
* Basic PowerPC Built-in Functions::
* PowerPC AltiVec/VSX Built-in Functions::
* PowerPC Hardware Transactional Memory Built-in Functions::
* PowerPC Atomic Memory Operation Functions::
* RX Built-in Functions::
* S/390 System z Built-in Functions::

I think it is nicer to just have a single "PowerPC Built-in Functions"
node, with then everything else hanging from a menu in there (as you have
for "Basic PowerPC Built-in Functions" now; so just rename that and put
the V*X, TM, atomic nodes in that menu too; probably need to make those
subsubsections or such).

> 2018-05-09  Kelvin Nilsen  <kel...@gcc.gnu.org>
> 
>       * doc/extend.texi (PowerPC Built-in Functions): Rename this
>       subsection.
>       (Basic PowerPC Built-in Functions): The new name of the
>       subsection previously known as "PowerPC Built-in Functions".
>       (Basic PowerPC Built-in Functions Available on all Configurations):
>       New subsubsection.
>       (Basic PowerPC Built-in Functions Available on ISA 2.05): Likewise.
>       (Basic PowerPC Built-in Functions Available on ISA 2.06): Likewise.
>       (Basic PowerPC Built-in Functions Available on ISA 2.07): Likewise.
>       (Basic PowerPC Built-in Functions Available on ISA 3.0): Likewise.

But please commit this now, it's a nice improvement already.  Thanks!


Segher

Reply via email to