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
Related show

Commit Message

Martin Jambor April 14, 2020, 2:17 p.m.
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(-)

-- 
2.26.0

Comments

Hongtao Liu via Gcc-patches April 14, 2020, 2:29 p.m. | #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. | #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

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