On Wed, 2025-03-12 at 12:09 +0100, Mark Wielaard wrote:
> Hi David,
> 
> On Mon, Mar 10, 2025 at 05:40:18PM -0400, David Malcolm wrote:
> > FWIW gcc/cobol/lang.opt.urls has some D-specific things that look
> > like
> > copy-and-paste cruft, but hopefully it won't cause problems.
> 
> And some Fortran stuff. The autoregen bot currently flags it with the
> following diff:
> 
> diff --git a/gcc/cobol/lang.opt.urls b/gcc/cobol/lang.opt.urls
> index a0e1f1944fe..6a5dc1c0f58 100644
> --- a/gcc/cobol/lang.opt.urls
> +++ b/gcc/cobol/lang.opt.urls
> @@ -10,20 +10,27 @@ UrlSuffix(gcc/Preprocessor-Options.html#index-D-
> 1)
>  I
>  UrlSuffix(gcc/Directory-Options.html#index-I)
> LangUrlSuffix_D(gdc/Directory-Options.html#index-I)
>  
> +ffixed-form
> +LangUrlSuffix_Fortran(gfortran/Fortran-Dialect-Options.html#index-
> ffixed-form)
> +
>  fsyntax-only
> -UrlSuffix(gcc/Warning-Options.html#index-fsyntax-only)
> LangUrlSuffix_D(gdc/Warnings.html#index-fno-syntax-only)
> +UrlSuffix(gcc/Warning-Options.html#index-fsyntax-only)
> LangUrlSuffix_D(gdc/Warnings.html#index-fno-syntax-only)
> LangUrlSuffix_Fortran(gfortran/Error-and-Warning-Options.html#index-
> fsyntax-only)
> +
> +ffree-form
> +LangUrlSuffix_Fortran(gfortran/Fortran-Dialect-Options.html#index-
> ffree-form)
>  
>  fmax-errors
>  UrlSuffix(gcc/Warning-Options.html#index-fmax-errors)
> LangUrlSuffix_D(gdc/Warnings.html#index-fmax-errors)
>  
>  iprefix
> -UrlSuffix(gcc/Directory-Options.html#index-iprefix)
> LangUrlSuffix_D(gdc/Directory-Options.html#index-iprefix)
> +UrlSuffix(gcc/Directory-Options.html#index-iprefix)
> LangUrlSuffix_D(gdc/Directory-Options.html#index-iprefix)
> LangUrlSuffix_Fortran(gfortran/Preprocessing-Options.html#index-
> iprefix)
>  
>  include
>  UrlSuffix(gcc/Preprocessor-Options.html#index-include)
>  
>  isysroot
> -UrlSuffix(gcc/Directory-Options.html#index-isysroot)
> +UrlSuffix(gcc/Directory-Options.html#index-isysroot)
> LangUrlSuffix_Fortran(gfortran/Preprocessing-Options.html#index-
> isysroot)
>  
>  isystem
> -UrlSuffix(gcc/Directory-Options.html#index-isystem)
> +UrlSuffix(gcc/Directory-Options.html#index-isystem)
> LangUrlSuffix_Fortran(gfortran/Preprocessing-Options.html#index-
> isystem)
> +
> 
> I don't really understand what these D and Fortran references are
> for. 

These eventually get processed by options-urls-cc-gen.awk; the dups
aren't a problem.


> Are these expected? Should we simply regenerate them with the
> above diff? 

Yes, and thanks for fixing this.

> Or does the make regenerate-opt-urls target need an
> update?
> 

Yes; I took a look at this, but PR web/119227 would need fixing first
(so that we have URLs to point at).

Dave

Reply via email to