Re: [PATCH] rs6000: Fix extraneous characters in the documentation
On Wed, Aug 19, 2020 at 04:03:31PM -0300, Tulio Magno Quites Machado Filho via Gcc-patches wrote: > Replace them with a whitespace in order to avoid artifacts in the HTML > document. Pushed to trunk now. Thanks! Segher
Re: [PATCH] rs6000: Fix extraneous characters in the documentation
Hi! On Wed, Aug 19, 2020 at 04:03:31PM -0300, Tulio Magno Quites Machado Filho via Gcc-patches wrote: > Replace them with a whitespace in order to avoid artifacts in the HTML > document. Multiple whitespaces in texinfo are supposed to work correctly, and are good to have for various reasons (more readable, and agrees to our coding conventions, and it agrees to texinfo's output, too!) But as Will says, there seem the be non-breaking space here, instead? 000: 3e20 2d62 656c 6f77 2069 7320 6465 6669 > -below is defi 010: 6e65 6420 746f 2062 6520 6120 6e6f 726d ned to be a norm 020: 616c 2076 6563 746f 7220 756e 7369 676e al vector unsign 030: 6564 2063 6861 7220 7479 7065 2ec2 a020 ed char type... 040: 5468 6520 7569 6e74 322c 2075 696e 7434 The uint2, uint4 050: 0a . 000: 3e20 2b62 656c 6f77 2069 7320 6465 6669 > +below is defi 010: 6e65 6420 746f 2062 6520 6120 6e6f 726d ned to be a norm 020: 616c 2076 6563 746f 7220 756e 7369 676e al vector unsign 030: 6564 2063 6861 7220 7479 7065 2e20 2054 ed char type. T 040: 6865 2075 696e 7432 2c20 7569 6e74 340a he uint2, uint4. Yeah, utf8 c2 a0, aka u+00a0. Replacing that with an actual space is of course okay (and trivial and obvious). Thanks! (Bonus points if you do all of doc/*.texi :-) ) Segher
Re: [PATCH] rs6000: Fix extraneous characters in the documentation
On Mon, 2020-10-05 at 17:23 -0300, Tulio Magno Quites Machado Filho via
Gcc-patches wrote:
> Ping?
+cc Segher :-)
>
> Tulio Magno Quites Machado Filho via Gcc-patches
> writes:
>
> > Replace them with a whitespace in order to avoid artifacts in the HTML
> > document.
> >
> > 2020-08-19 Tulio Magno Quites Machado Filho
> >
> > gcc/
> > * doc/extend.texi (PowerPC Built-in Functions): Replace
> > extraneous characters with whitespace.
> > ---
> > gcc/doc/extend.texi | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
> > index bcc251481ca..0c380322280 100644
> > --- a/gcc/doc/extend.texi
> > +++ b/gcc/doc/extend.texi
> > @@ -21538,10 +21538,10 @@ void amo_stdat_smin (int64_t *, int64_t);
> > ISA 3.1 of the PowerPC added new Matrix-Multiply Assist (MMA) instructions.
> > GCC provides support for these instructions through the following built-in
> > functions which are enabled with the @code{-mmma} option. The vec_t type
> > -below is defined to be a normal vector unsigned char type. The uint2,
> > uint4
> > +below is defined to be a normal vector unsigned char type. The uint2,
> > uint4
That looks like a non-breaking space. (ascii c2 a0) so
2e c2 a0 20 becomes 2e 20 20
> > and uint8 parameters are 2-bit, 4-bit and 8-bit unsigned integer constants
> > -respectively. The compiler will verify that they are constants and that
> > -their values are within range.
> > +respectively. The compiler will verify that they are constants and that
> > +their values are within range.
2e c2 a0 20 becomes 2e 20 20
And drops a trailing whitespace.
Those seem reasonable.
lgtm
Thanks
-Will
> >
> > The built-in functions supported are:
> >
> > --
> > 2.25.4
> >
>
>
Re: [PATCH] rs6000: Fix extraneous characters in the documentation
Ping?
Tulio Magno Quites Machado Filho via Gcc-patches
writes:
> Replace them with a whitespace in order to avoid artifacts in the HTML
> document.
>
> 2020-08-19 Tulio Magno Quites Machado Filho
>
> gcc/
> * doc/extend.texi (PowerPC Built-in Functions): Replace
> extraneous characters with whitespace.
> ---
> gcc/doc/extend.texi | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
> index bcc251481ca..0c380322280 100644
> --- a/gcc/doc/extend.texi
> +++ b/gcc/doc/extend.texi
> @@ -21538,10 +21538,10 @@ void amo_stdat_smin (int64_t *, int64_t);
> ISA 3.1 of the PowerPC added new Matrix-Multiply Assist (MMA) instructions.
> GCC provides support for these instructions through the following built-in
> functions which are enabled with the @code{-mmma} option. The vec_t type
> -below is defined to be a normal vector unsigned char type. The uint2, uint4
> +below is defined to be a normal vector unsigned char type. The uint2, uint4
> and uint8 parameters are 2-bit, 4-bit and 8-bit unsigned integer constants
> -respectively. The compiler will verify that they are constants and that
> -their values are within range.
> +respectively. The compiler will verify that they are constants and that
> +their values are within range.
>
> The built-in functions supported are:
>
> --
> 2.25.4
>
--
Tulio Magno
[PATCH] rs6000: Fix extraneous characters in the documentation
Replace them with a whitespace in order to avoid artifacts in the HTML
document.
2020-08-19 Tulio Magno Quites Machado Filho
gcc/
* doc/extend.texi (PowerPC Built-in Functions): Replace
extraneous characters with whitespace.
---
gcc/doc/extend.texi | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
index bcc251481ca..0c380322280 100644
--- a/gcc/doc/extend.texi
+++ b/gcc/doc/extend.texi
@@ -21538,10 +21538,10 @@ void amo_stdat_smin (int64_t *, int64_t);
ISA 3.1 of the PowerPC added new Matrix-Multiply Assist (MMA) instructions.
GCC provides support for these instructions through the following built-in
functions which are enabled with the @code{-mmma} option. The vec_t type
-below is defined to be a normal vector unsigned char type. The uint2, uint4
+below is defined to be a normal vector unsigned char type. The uint2, uint4
and uint8 parameters are 2-bit, 4-bit and 8-bit unsigned integer constants
-respectively. The compiler will verify that they are constants and that
-their values are within range.
+respectively. The compiler will verify that they are constants and that
+their values are within range.
The built-in functions supported are:
--
2.25.4
