This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Ping #1: [Patch,AVR]: Auto-generate all -mmcu= options in documentation.


Denis Chertykov schrieb:

This patch adds a new file ./gcc/doc/avr-mmcu.texi that lists all valid
-mmcu= settings and replaces the respective text in invoke.texi by
@include avr-mmcu.texi

Up to now, there is no complete list of -mmcu= options, and a list is
hard to maintain by hand because it contains more than 180 devices.

If, during the build of avr-gcc, a change of ./gcc/doc/avr-mmcu.texi
is detected, the build aborts with a message that nags the user to
copy the new content of avr-mmcu.texi to ./gcc/doc/avr-mmcu.texi.

The error message's text is:

*** Verify that you have permission to grant a
*** GFDL license for all new text in
*** avr-mmcu.texi, then copy it to $(srcdir)/doc/avr-mmcu.texi

./gcc/doc/avr-mmcu.texi is auto-generated, but there is no rule to
automatically update it during the build process.

Documents (HTML, PDF, ...) build fine.

Ok for trunk?

If it's appropriate for 4.7, I'd change invoke.texi accordingly by
copy-pasting the auto-generated texi code into that file, i.e.
into section "AVR Options".

Johann

* Makefile.in (TEXI_GCC_FILES): Add avr-mmcu.texi.

      * doc/avr-mmcu.texi: New auto-generated file.
      * doc/invoke.texi (AVR Options): Include avr-mmcu.texi in order
      to document all valid -mmcu= arguments.

      * config/avr/avr.h (arch_info_s): New struct definition.
      * config/avr/avr-devices.c (avr_texinfo): New variable.
      * config/avr/gen-avr-mmcu-texi.c: New file.
      * config/avr/t-avr: New rules and dependencies to build avr-mmcu.texi.


AVR port  modifications approved but I think that somebody else must
approve your changes to Makefile.in and doc/*

http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00961.html


There is approval missing from a global reviewer for the following changes:

* Makefile.in (TEXI_GCC_FILES): Add avr-mmcu.texi.

       * doc/avr-mmcu.texi: New auto-generated file.
       * doc/invoke.texi (AVR Options): Include avr-mmcu.texi in order
       to document all valid -mmcu= arguments.

Johann


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]