doc: explicitly say 'lifetime' for DCE

Message ID 20231102090234.1145382-1-sam@gentoo.org
State Accepted
Headers
Series doc: explicitly say 'lifetime' for DCE |

Checks

Context Check Description
snail/gcc-patch-check success Github commit url

Commit Message

Sam James Nov. 2, 2023, 9:02 a.m. UTC
  Say 'memory lifetime' rather than 'memory life' as lifetime is the more
standard term nowadays (indeed we have e.g. -fno-lifetime-dse).

It's also easier to grep for if someone is looking for the documentation on
where we do that.

gcc/ChangeLog:
    * doc/passes.texi (Dead code elimination): Explicitly say 'lifetime'
    as this has become the standard term for what we're doing here.

Signed-off-by: Sam James <sam@gentoo.org>
---
 gcc/doc/passes.texi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Richard Biener Nov. 2, 2023, 10:18 a.m. UTC | #1
On Thu, Nov 2, 2023 at 10:03 AM Sam James <sam@gentoo.org> wrote:
>
> Say 'memory lifetime' rather than 'memory life' as lifetime is the more
> standard term nowadays (indeed we have e.g. -fno-lifetime-dse).
>
> It's also easier to grep for if someone is looking for the documentation on
> where we do that.

OK

> gcc/ChangeLog:
>     * doc/passes.texi (Dead code elimination): Explicitly say 'lifetime'
>     as this has become the standard term for what we're doing here.
>
> Signed-off-by: Sam James <sam@gentoo.org>
> ---
>  gcc/doc/passes.texi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/doc/passes.texi b/gcc/doc/passes.texi
> index eb2bb6062834..470ac498a132 100644
> --- a/gcc/doc/passes.texi
> +++ b/gcc/doc/passes.texi
> @@ -543,7 +543,7 @@ and is defined by @code{pass_early_warn_uninitialized} and
>  @item Dead code elimination
>
>  This pass scans the function for statements without side effects whose
> -result is unused.  It does not do memory life analysis, so any value
> +result is unused.  It does not do memory lifetime analysis, so any value
>  that is stored in memory is considered used.  The pass is run multiple
>  times throughout the optimization process.  It is located in
>  @file{tree-ssa-dce.cc} and is described by @code{pass_dce}.
> --
> 2.42.0
>
  
Sam James Nov. 2, 2023, 10:25 a.m. UTC | #2
Richard Biener <richard.guenther@gmail.com> writes:

> On Thu, Nov 2, 2023 at 10:03 AM Sam James <sam@gentoo.org> wrote:
>>
>> Say 'memory lifetime' rather than 'memory life' as lifetime is the more
>> standard term nowadays (indeed we have e.g. -fno-lifetime-dse).
>>
>> It's also easier to grep for if someone is looking for the documentation on
>> where we do that.
>
> OK

Could you push for me please? I have a sw account but no gcc access
(yet).

cheers

>
>> gcc/ChangeLog:
>>     * doc/passes.texi (Dead code elimination): Explicitly say 'lifetime'
>>     as this has become the standard term for what we're doing here.
>>
>> Signed-off-by: Sam James <sam@gentoo.org>
>> ---
>>  gcc/doc/passes.texi | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/gcc/doc/passes.texi b/gcc/doc/passes.texi
>> index eb2bb6062834..470ac498a132 100644
>> --- a/gcc/doc/passes.texi
>> +++ b/gcc/doc/passes.texi
>> @@ -543,7 +543,7 @@ and is defined by @code{pass_early_warn_uninitialized} and
>>  @item Dead code elimination
>>
>>  This pass scans the function for statements without side effects whose
>> -result is unused.  It does not do memory life analysis, so any value
>> +result is unused.  It does not do memory lifetime analysis, so any value
>>  that is stored in memory is considered used.  The pass is run multiple
>>  times throughout the optimization process.  It is located in
>>  @file{tree-ssa-dce.cc} and is described by @code{pass_dce}.
>> --
>> 2.42.0
>>
  
Richard Biener Nov. 2, 2023, 11:46 a.m. UTC | #3
On Thu, Nov 2, 2023 at 11:25 AM Sam James <sam@gentoo.org> wrote:
>
>
> Richard Biener <richard.guenther@gmail.com> writes:
>
> > On Thu, Nov 2, 2023 at 10:03 AM Sam James <sam@gentoo.org> wrote:
> >>
> >> Say 'memory lifetime' rather than 'memory life' as lifetime is the more
> >> standard term nowadays (indeed we have e.g. -fno-lifetime-dse).
> >>
> >> It's also easier to grep for if someone is looking for the documentation on
> >> where we do that.
> >
> > OK
>
> Could you push for me please? I have a sw account but no gcc access
> (yet).

Done after fixing ChangeLog format.

Richard.

> cheers
>
> >
> >> gcc/ChangeLog:
> >>     * doc/passes.texi (Dead code elimination): Explicitly say 'lifetime'
> >>     as this has become the standard term for what we're doing here.
> >>
> >> Signed-off-by: Sam James <sam@gentoo.org>
> >> ---
> >>  gcc/doc/passes.texi | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/gcc/doc/passes.texi b/gcc/doc/passes.texi
> >> index eb2bb6062834..470ac498a132 100644
> >> --- a/gcc/doc/passes.texi
> >> +++ b/gcc/doc/passes.texi
> >> @@ -543,7 +543,7 @@ and is defined by @code{pass_early_warn_uninitialized} and
> >>  @item Dead code elimination
> >>
> >>  This pass scans the function for statements without side effects whose
> >> -result is unused.  It does not do memory life analysis, so any value
> >> +result is unused.  It does not do memory lifetime analysis, so any value
> >>  that is stored in memory is considered used.  The pass is run multiple
> >>  times throughout the optimization process.  It is located in
> >>  @file{tree-ssa-dce.cc} and is described by @code{pass_dce}.
> >> --
> >> 2.42.0
> >>
>
  
Sam James Nov. 2, 2023, 12:01 p.m. UTC | #4
Richard Biener <richard.guenther@gmail.com> writes:

> On Thu, Nov 2, 2023 at 11:25 AM Sam James <sam@gentoo.org> wrote:
>>
>>
>> Richard Biener <richard.guenther@gmail.com> writes:
>>
>> > On Thu, Nov 2, 2023 at 10:03 AM Sam James <sam@gentoo.org> wrote:
>> >>
>> >> Say 'memory lifetime' rather than 'memory life' as lifetime is the more
>> >> standard term nowadays (indeed we have e.g. -fno-lifetime-dse).
>> >>
>> >> It's also easier to grep for if someone is looking for the documentation on
>> >> where we do that.
>> >
>> > OK
>>
>> Could you push for me please? I have a sw account but no gcc access
>> (yet).
>
> Done after fixing ChangeLog format.
>
> Richard.

Thanks!

>
>> cheers
>>
>> >
>> >> gcc/ChangeLog:
>> >>     * doc/passes.texi (Dead code elimination): Explicitly say 'lifetime'
>> >>     as this has become the standard term for what we're doing here.
>> >>
>> >> Signed-off-by: Sam James <sam@gentoo.org>
>> >> ---
>> >>  gcc/doc/passes.texi | 2 +-
>> >>  1 file changed, 1 insertion(+), 1 deletion(-)
>> >>
>> >> diff --git a/gcc/doc/passes.texi b/gcc/doc/passes.texi
>> >> index eb2bb6062834..470ac498a132 100644
>> >> --- a/gcc/doc/passes.texi
>> >> +++ b/gcc/doc/passes.texi
>> >> @@ -543,7 +543,7 @@ and is defined by @code{pass_early_warn_uninitialized} and
>> >>  @item Dead code elimination
>> >>
>> >>  This pass scans the function for statements without side effects whose
>> >> -result is unused.  It does not do memory life analysis, so any value
>> >> +result is unused.  It does not do memory lifetime analysis, so any value
>> >>  that is stored in memory is considered used.  The pass is run multiple
>> >>  times throughout the optimization process.  It is located in
>> >>  @file{tree-ssa-dce.cc} and is described by @code{pass_dce}.
>> >> --
>> >> 2.42.0
>> >>
>>
  

Patch

diff --git a/gcc/doc/passes.texi b/gcc/doc/passes.texi
index eb2bb6062834..470ac498a132 100644
--- a/gcc/doc/passes.texi
+++ b/gcc/doc/passes.texi
@@ -543,7 +543,7 @@  and is defined by @code{pass_early_warn_uninitialized} and
 @item Dead code elimination
 
 This pass scans the function for statements without side effects whose
-result is unused.  It does not do memory life analysis, so any value
+result is unused.  It does not do memory lifetime analysis, so any value
 that is stored in memory is considered used.  The pass is run multiple
 times throughout the optimization process.  It is located in
 @file{tree-ssa-dce.cc} and is described by @code{pass_dce}.