diff mbox series

wwwdocs: Slightly improve IPA-SRA description in gcc-10/changes.html

Message ID ri68siyywvv.fsf@suse.cz
State New
Headers show
Series wwwdocs: Slightly improve IPA-SRA description in gcc-10/changes.html | expand

Commit Message

Martin Jambor April 14, 2020, 2:17 p.m. UTC
Hi,

I think that the fact that IPA-SRA can now remove unused return
values deserves a special mention - and I also fixed the proposition
in the name of the pass.

OK for wwwdocs?

Thanks,

Martin


---
 htdocs/gcc-10/changes.html | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Richard Biener April 14, 2020, 2:29 p.m. UTC | #1
On Tue, Apr 14, 2020 at 4:19 PM Martin Jambor <mjambor@suse.cz> wrote:
>
> Hi,
>
> I think that the fact that IPA-SRA can now remove unused return
> values deserves a special mention - and I also fixed the proposition
> in the name of the pass.
>
> OK for wwwdocs?
>
> Thanks,
>
> Martin
>
>
> ---
>  htdocs/gcc-10/changes.html | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html
> index 17679812..1e3aca18 100644
> --- a/htdocs/gcc-10/changes.html
> +++ b/htdocs/gcc-10/changes.html
> @@ -103,8 +103,9 @@ a work-in-progress.</p>
>    <li>
>      Inter-procedural optimization improvements:
>      <ul>
> -      <li>The inter-procedural scalar replacement for aggregates (IPA-SRA) pass
> -         was re-implemented to work at link-time.
> +      <li>The inter-procedural scalar replacement of aggregates (IPA-SRA) pass
> +         was re-implemented to work at link-time and can now also remove
> +         calculating and passing unused return values.

and returning

OK with that change.

>        </li>
>        <li><a href="https://gcc.gnu.org/onlinedocs/gcc-10.1.0/gcc/Optimize-Options.html#index-finline-functions"><code>-finline-functions</code></a>
>           is now enabled at <code>-O2</code> and was retuned for better code
> --
> 2.26.0
>
Gerald Pfeifer April 14, 2020, 2:59 p.m. UTC | #2
On Tue, 14 Apr 2020, Martin Jambor wrote:
> I think that the fact that IPA-SRA can now remove unused return
> values deserves a special mention - and I also fixed the proposition
> in the name of the pass.
> 
> OK for wwwdocs?

Yes, thank you.  A small recommendation:

> +      <li>The inter-procedural scalar replacement of aggregates (IPA-SRA) pass
> +	  was re-implemented to work at link-time and can now also remove
> +	  calculating and passing unused return values.

Here I'd use "computing" instead of "calculating".

The meanings of those two words overlap, and in the sense of formal
logics everything could be seen as a calculation.  In the more common
term 

  x = 47*(y+1)

would be more of a calculation whereas

  if (y > 47) x = y; else x = f(y);

more of a computation.

Gerald
diff mbox series

Patch

diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html
index 17679812..1e3aca18 100644
--- a/htdocs/gcc-10/changes.html
+++ b/htdocs/gcc-10/changes.html
@@ -103,8 +103,9 @@  a work-in-progress.</p>
   <li>
     Inter-procedural optimization improvements:
     <ul>
-      <li>The inter-procedural scalar replacement for aggregates (IPA-SRA) pass
-	  was re-implemented to work at link-time.
+      <li>The inter-procedural scalar replacement of aggregates (IPA-SRA) pass
+	  was re-implemented to work at link-time and can now also remove
+	  calculating and passing unused return values.
       </li>
       <li><a href="https://gcc.gnu.org/onlinedocs/gcc-10.1.0/gcc/Optimize-Options.html#index-finline-functions"><code>-finline-functions</code></a>
 	  is now enabled at <code>-O2</code> and was retuned for better code