Re: [Doc, AArch64] Fix/Update AArch64 options.

[Sandra Loosemore <sandra at codesourcery dot com>]

On 06/27/2017 06:19 AM, Yvan Roux wrote:

> diff --git a/gcc/config/aarch64/aarch64.opt b/gcc/config/aarch64/aarch64.opt
> index 942a7d5..0fd1bfa 100644
> --- a/gcc/config/aarch64/aarch64.opt
> +++ b/gcc/config/aarch64/aarch64.opt
> @@ -146,7 +146,7 @@ EnumValue
>  Enum(aarch64_abi) String(lp64) Value(AARCH64_ABI_LP64)
>
>  mpc-relative-literal-loads
> -Target Report Save Var(pcrelative_literal_loads) Init(2) Save
> +Target Report Var(pcrelative_literal_loads) Init(2) Save
>  PC relative literal loads.
>
>  msign-return-address=

I think this qualifies as an obvious fix.  I can't approve it if it 
isn't, anyway....  ;-)

> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> index d1e097b..6e0e776 100644
> --- a/gcc/doc/invoke.texi
> +++ b/gcc/doc/invoke.texi
> @@ -595,7 +595,9 @@ Objective-C and Objective-C++ Dialects}.
>  -mlow-precision-recip-sqrt  -mno-low-precision-recip-sqrt@gol
>  -mlow-precision-sqrt  -mno-low-precision-sqrt@gol
>  -mlow-precision-div  -mno-low-precision-div @gol
> --march=@var{name}  -mcpu=@var{name}  -mtune=@var{name}}
> +-mpc-relative-literal-loads -mno-pc-relative-literal-loads @gol

For options that have both positive and negative variants, we should 
only be listing the one that is not the default in the Option Summary 
table.  Can you please remove the existing redundant options listed for 
AArch64, instead of adding a new one?

> +-msign-return-address=@var{scope} @gol
> +-march=@var{name}  -mcpu=@var{name}  -mtune=@var{name}  -moverride=@var{string}}
>
>  @emph{Adapteva Epiphany Options}
>  @gccoptlist{-mhalf-reg-file  -mprefer-short-insn-regs @gol
> @@ -14158,8 +14160,10 @@ across releases.
>  This option is only intended to be useful when developing GCC.
>
>  @item -mpc-relative-literal-loads
> +@item -mno-pc-relative-literal-loads

It is OK to list both the positive and negative forms in the full 
description, but in a table with multiple items in the same entry, the 
second and subsequent ones should use @itemx markup instead of @item.

>  @opindex mpc-relative-literal-loads
> -Enable PC-relative literal loads.  With this option literal pools are
> +@opindex mno-pc-relative-literal-loads
> +Enable or disable PC-relative literal loads.  With this option literal pools are
>  accessed using a single instruction and emitted after each function.  This
>  limits the maximum size of functions to 1MB.  This is enabled by default for
>  @option{-mcmodel=tiny}.

-Sandra