On Thu, Aug 27, 2026 at 8:35 AM Jakub Jelinek <[email protected]> wrote:
>
> On Thu, Aug 27, 2026 at 01:46:39PM +0200, Richard Biener wrote:
> > > I think there should be 2 spaces before P, but we aren't consistent:
> > > grep 'nothing.*Preserved for' *.opt */*.opt | sort | uniq -c | sort -n
> > >       1 fortran/lang.opt:Does nothing.  Preserved for backward 
> > > compatibility.
> > >       2 analyzer/analyzer.opt:Does nothing. Preserved for backward 
> > > compatibility.
> > >       2 c-family/c.opt:Does nothing. Preserved for backward compatibility.
> > >       2 fortran/lang.opt:Does nothing. Preserved for backward 
> > > compatibility.
> > >       4 c-family/c.opt:Does nothing.  Preserved for backward 
> > > compatibility.
> > >      13 common.opt:Does nothing. Preserved for backward compatibility.
> > >      35 common.opt:Does nothing.  Preserved for backward compatibility.
> >
> > Thanks, I fixed that and the case for the immediately following
> > ftree-loop-if-convert-stores.
>
> Here is a patch to fix the cases except that one.
>
> Tested on x86_64-linux, ok for trunk?

Ok.

>
> 2026-08-27  Jakub Jelinek  <[email protected]>
>
> gcc/
>         * common.opt (Wunsafe-loop-optimizations): Use 2 spaces instead of
>         1 between sentences in the description.
>         (Wunreachable-code, fargument-alias, fargument-noalias,
>         fargument-noalias-global, fargument-noalias-anything,
>         floop-flatten, fipa-matrix-reorg, fipa-struct-reorg,
>         foptimize-register-move, fregmove, ftree-vect-loop-version):
>         Likewise.
>         * config/alpha/alpha.opt (mgas): Use 2 spaces instead of 1 between
>         sentences in the description.
> gcc/analyzer/
>         * analyzer.opt (fanalyzer-fine-grained): Use 2 spaces instead of 1
>         between sentences in the description.
>         (fdump-analyzer-callgraph): Likewise.
> gcc/c-family/
>         * c.opt (Wsynth): Use 2 spaces instead of 1 between
>         sentences in the description.
>         (fallow-parameterless-variadic-functions): Likewise.
> gcc/fortran/
>         * lang.opt (Wargument-mismatch): Use 2 spaces instead of 1 between
>         sentences in the description.
>         (fdump-core): Likewise.
>
> --- a/gcc/common.opt    2026-08-20 12:34:01.299794811 +0200
> +++ b/gcc/common.opt    2026-08-27 15:39:42.277227193 +0200
> @@ -696,7 +696,7 @@ Warn if dereferencing a NULL pointer may
>
>  Wunsafe-loop-optimizations
>  Common Ignore Warning
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  Wmissing-noreturn
>  Common Warning Alias(Wsuggest-attribute=noreturn)
> @@ -854,7 +854,7 @@ Warn about maybe uninitialized automatic
>
>  Wunreachable-code
>  Common Ignore Warning
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  Wunused
>  Common Var(warn_unused) Init(0) Warning
> @@ -1134,19 +1134,19 @@ Enable static analysis pass.
>
>  fargument-alias
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fargument-noalias
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fargument-noalias-global
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fargument-noalias-anything
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fsanitize=
>  Common Driver Joined
> @@ -1938,7 +1938,7 @@ Use STB_GNU_UNIQUE if supported by the a
>
>  floop-flatten
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  floop-nest-optimize
>  Common Var(flag_loop_nest_optimize) Optimization
> @@ -2225,11 +2225,11 @@ Reduce stack alignment on call sites if
>
>  fipa-matrix-reorg
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fipa-struct-reorg
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fipa-vrp
>  Common Var(flag_ipa_vrp) Optimization
> @@ -2567,7 +2567,7 @@ If neither case applies, generate table-
>
>  foptimize-register-move
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  foptimize-sibling-calls
>  Common Var(flag_optimize_sibling_calls) Optimization
> @@ -2793,7 +2793,7 @@ Return small aggregates in registers.
>
>  fregmove
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  flifetime-dse
>  Common Var(flag_lifetime_dse,2) Init(2) Optimization
> @@ -3574,7 +3574,7 @@ Enables the dynamic vectorizer cost mode
>
>  ftree-vect-loop-version
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  ftree-scev-cprop
>  Common Var(flag_tree_scev_cprop) Init(1) Optimization
> --- a/gcc/config/alpha/alpha.opt        2026-03-27 10:17:13.774336614 +0100
> +++ b/gcc/config/alpha/alpha.opt        2026-08-27 15:40:09.898865335 +0200
> @@ -28,7 +28,7 @@ Use fp registers.
>
>  mgas
>  Target Ignore Undocumented
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  mieee-conformant
>  Target RejectNegative Mask(IEEE_CONFORMANT)
> --- a/gcc/analyzer/analyzer.opt 2026-05-19 09:11:47.993259739 +0200
> +++ b/gcc/analyzer/analyzer.opt 2026-08-27 15:38:37.630076744 +0200
> @@ -308,7 +308,7 @@ Add extra annotations to diagrams.
>
>  fanalyzer-fine-grained
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fanalyzer-feasibility
>  Common Var(flag_analyzer_feasibility) Init(1)
> @@ -372,7 +372,7 @@ Dump various analyzer internals to stder
>
>  fdump-analyzer-callgraph
>  Common Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fdump-analyzer-exploded-graph
>  Common RejectNegative Var(flag_dump_analyzer_exploded_graph)
> --- a/gcc/c-family/c.opt        2026-07-13 18:37:33.594239874 +0200
> +++ b/gcc/c-family/c.opt        2026-08-27 15:39:01.013769451 +0200
> @@ -1522,7 +1522,7 @@ Warn when __sync_fetch_and_nand and __sy
>
>  Wsynth
>  C++ ObjC++ Warning Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  Wsystem-headers
>  C ObjC C++ ObjC++ Warning
> @@ -1743,7 +1743,7 @@ C++ ObjC++ WarnRemoved
>
>  fallow-parameterless-variadic-functions
>  C ObjC Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  falt-external-templates
>  C++ ObjC++ WarnRemoved
> --- a/gcc/fortran/lang.opt      2026-06-12 16:55:06.684171344 +0200
> +++ b/gcc/fortran/lang.opt      2026-08-27 15:40:25.256664455 +0200
> @@ -211,7 +211,7 @@ Warn about creation of array temporaries
>
>  Wargument-mismatch
>  Fortran WarnRemoved
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  Wc-binding-type
>  Fortran Var(warn_c_binding_type) Warning LangEnabledBy(Fortran,Wall)
> @@ -560,7 +560,7 @@ Allow dollar signs in entity names.
>
>  fdump-core
>  Fortran Ignore
> -Does nothing. Preserved for backward compatibility.
> +Does nothing.  Preserved for backward compatibility.
>
>  fdump-fortran-original
>  Fortran Var(flag_dump_fortran_original)
>
>
>         Jakub
>

Reply via email to