[PATCH] Show valid options for -march and -mtune in --help=target for arm32 (PR driver/83193).

Thomas Preudhomme thomas.preudhomme@linaro.org
Wed Jul 18 16:28:00 GMT 2018


Hi Martin,

Why is this needed when -mfpu does not seem to need it for instance?
Regarding the patch:

> -    print "Name(processor_type) Type(enum processor_type)"
> -    print "Known ARM CPUs (for use with the -mcpu= and -mtune= options):\n"
> +    print "Name(processor_type) Type(enum processor_type) ForceHelp"
> +    print "Known ARM CPUs (for use with the -mtune= options):\n"

Why changing the text beyond adding ForceHelp?

> +@item ForceHelp
> +This property is optional.  If present, enum values is printed
> +in @option{--help} output.
> +

are printed

Thanks,

Thomas
On Wed, 18 Jul 2018 at 16:50, Martin Liška <mliska@suse.cz> wrote:
>
> Hi.
>
> This introduces new ForceHelp option flag that helps to
> print valid option enum values that are not directly
> used as a type of an option.
>
> May I please ask ARM folks to test the patch?
> Thanks,
> Martin
>
> gcc/ChangeLog:
>
> 2018-07-18  Martin Liska  <mliska@suse.cz>
>
>         PR driver/83193
>         * config/arm/arm-tables.opt: Add ForceHelp flag for
>         processor_type and arch_name enum types.
>         * config/arm/parsecpu.awk: Likewise.
>         * doc/options.texi: Document new flag ForceHelp.
>         * opt-read.awk: Parse ForceHelp and set it in construction.
>         * optc-gen.awk: Likewise.
>         * opts.c (print_filtered_help): Handle force_help option.
>         * opts.h (struct cl_enum): New field force_help.
> ---
>  gcc/config/arm/arm-tables.opt | 6 +++---
>  gcc/config/arm/parsecpu.awk   | 6 +++---
>  gcc/doc/options.texi          | 4 ++++
>  gcc/opt-read.awk              | 3 +++
>  gcc/optc-gen.awk              | 3 ++-
>  gcc/opts.c                    | 3 ++-
>  gcc/opts.h                    | 3 +++
>  7 files changed, 20 insertions(+), 8 deletions(-)
>
>



More information about the Gcc-patches mailing list