On Thu, Oct 31, 2024 at 04:05:30PM -0400, James K. Lowden wrote:
> >From ccb8a64c97461d792fdc39db249980a9ecb63b4bbld.patch 4 Oct 2024 12:01:22
> >-0400
> From: "James K. Lowden" <[email protected]>
> Date: Fri 01 Nov 2024 01:50:33 PM EDT
> Subject: [PATCH] Add 'cobol' to 10 files
>
> ChangeLog
> * Makefile.def: Add libgcobol module and cobol language.
> * configure: Regenerated
> * configure.ac: Add libgcobol module and cobol language.
>
> gcc/ChangeLog
> * common.opt: Add libgcobol module and cobol language.
> * dwarf2out.cc: Add libgcobol module and cobol language
>
> gcc/cobol/ChangeLog
> * LICENSE: Add LICENSE
> * Make-lang.in: Add Make-lang.in
> * config-lang.in: Add config-lang.in
> * lang.opt: Add lang.opt
> * lang.opt.urls: Add lang.opt.urls
First of all, I think the libgcobol directory needs to be added before this
is committed, so that means the first patch should add gcc/cobol and
libgcobol as new ChangeLog locations, the gcc/cobol/ChangeLog addition patch
should also add libgcobol/ChangeLog, then some patch should add libgcobol
and gcc/cobol content and only the last step would be to wire it up in
toplevel etc.
As for above mentioned ChangeLog entries:
1) all descriptions should start with capital letter (you do that) and
end with . (only 3 out of 10 descriptions do that)
2) if you add a new file, the description should be
: New file.
or (for tests)
: New test.
Not Add <the name of the file again>
3) Regenerate. rather than Regenerated.
4) where appropriate, you should mention what you exactly changed.
So
* common.opt (static-libgcobol): New option.
* dwarf2out.cc (gen_compile_unit_die): Set language to DW_LANG_Cobol85
for Cobol. Add appropriate DW_AT_identifier_case for it.
or
* Makefile.def (target_modules): Add libgcobol.
(flags_to_pass): Add GCOBOL_FOR_TARGET.
(lang_env_dependencies): Add libgcobol entry.
(dependencies): Let install-target-libgcobol depend on
install-target-libstdc++-v3.
(languages): Add cobol.
* configure.ac (target_libraries): Add target-libgcobol.
> +# Copyright (c) 2019-2022 Symas Corporation
Surely just -2022?
> +# Copyright (C) 2016 Free Software Foundation, Inc.
Should be 2016-2024
> --- /dev/null
> +++ b/gcc/cobol/lang.opt.urls
> @@ -0,0 +1,29 @@
> +; Copied by Dubner from gcc/rust/ so that compilation could proceed
> +; Autogenerated by regenerate-opt-urls.py from gcc/rust/lang.opt and
> generated HTML
This file shouldn't be copied but generated.
make regenerate-opt-urls
> --- a/gcc/dwarf2out.cc
> +++ b/gcc/dwarf2out.cc
> @@ -25391,6 +25391,8 @@ gen_compile_unit_die (const char *filename)
> }
> else if (strcmp (language_string, "GNU F77") == 0)
> language = DW_LANG_Fortran77;
> + else if (strcmp (language_string, "Cobol") == 0)
> + language = DW_LANG_Cobol85;
> else if (strcmp (language_string, "GNU Modula-2") == 0)
> language = DW_LANG_Modula2;
> else if (dwarf_version >= 3 || !dwarf_strict)
> @@ -25442,6 +25444,9 @@ gen_compile_unit_die (const char *filename)
> lowercases everything. */
> add_AT_unsigned (die, DW_AT_identifier_case, DW_ID_down_case);
> break;
> + case DW_LANG_Cobol85:
> + add_AT_unsigned (die, DW_AT_identifier_case, DW_ID_case_insensitive);
Missing break;
Jakub
