[Bug other/100826] New: Add that "-fgcse-after-reload" is enabled at "-O3"

[Bug other/100826] New: Add that "-fgcse-after-reload" is enabled at "-O3"

[firasuke at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100826

            Bug ID: 100826
           Summary: Add that "-fgcse-after-reload" is enabled at "-O3"
           Product: gcc
           Version: 11.1.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
          Assignee: unassigned at gcc dot gnu.org
          Reporter: firasuke at gmail dot com
  Target Milestone: ---

Hello there,

Upon reading the GCC manual I stumbled upon the following flag
`-fgcse-after-reload`, which happens to be mentioned twice, the first time
being at:

```
       -O3 Optimize yet more.  -O3 turns on all optimizations specified by -O2
and also turns on the following optimization flags:

           -fgcse-after-reload -fipa-cp-clone ...
```

and the second time:

```
       -fgcse-after-reload
           When -fgcse-after-reload is enabled, a redundant load elimination
pass is performed after reload.  The purpose of this pass is to clean up
redundant
           spilling.

           Enabled by -fprofile-use and -fauto-profile.
```

I think it'd be better if the following statement:

"Enabled by -fprofile-use and -fauto-profile."

be changed to:

"Enabled by -O3, -fprofile-use, and -fauto-profile."

to show that it is also enabled by `-O3`, similar to `-fpeel-loops`.

Thanks in advance!

[Bug other/100826] Add that "-fgcse-after-reload" is enabled at "-O3"

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100826

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2021-05-31
           Assignee|unassigned at gcc dot gnu.org      |marxin at gcc dot gnu.org
     Ever confirmed|0                           |1
                 CC|                            |marxin at gcc dot gnu.org
             Status|UNCONFIRMED                 |ASSIGNED

--- Comment #1 from Martin Liška <marxin at gcc dot gnu.org> ---
I'll take a look. Thanks for the report.

[Bug other/100826] Add that "-fgcse-after-reload" is enabled at "-O3"

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100826

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Liska <marxin@gcc.gnu.org>:

https://gcc.gnu.org/g:fdbd0cb73af527f0630f0cbb26edb8584f593fea

commit r12-1140-gfdbd0cb73af527f0630f0cbb26edb8584f593fea
Author: Martin Liska <mliska@suse.cz>
Date:   Tue Jun 1 14:17:39 2021 +0200

    docs: Mention that -fgcse-after-reload is enabled with -O3.

    gcc/ChangeLog:

            PR other/100826
            * doc/invoke.texi: Mention that -fgcse-after-reload
            is enabled with -O3.

[Bug other/100826] Add that "-fgcse-after-reload" is enabled at "-O3"

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100826

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #3 from Martin Liška <marxin at gcc dot gnu.org> ---
Fixed.

[Bug other/100826] Add that "-fgcse-after-reload" is enabled at "-O3"

[firasuke at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100826

--- Comment #4 from Firas Khalil Khana <firasuke at gmail dot com> ---
Glad I could help. Thanks for your time and effort!

[Bug other/100826] Add that "-fgcse-after-reload" is enabled at "-O3"

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100826

--- Comment #5 from Martin Liška <marxin at gcc dot gnu.org> ---
(In reply to Firas Khalil Khana from comment #4)
> Glad I could help. Thanks for your time and effort!

You're welcome. We thank you for reporting that!

[Bug other/100826] Add that "-fgcse-after-reload" is enabled at "-O3"

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100826

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |documentation
   Target Milestone|---                         |12.0