Document -fexcess-precision=16 in tm.texi

Message ID 20220909094640.24077-1-palmer@rivosinc.com
State Committed
Commit 4dcd340ebaae8213eeae89894b5ecdf61b0eb100
Headers
Series Document -fexcess-precision=16 in tm.texi |

Commit Message

Palmer Dabbelt Sept. 9, 2022, 9:46 a.m. UTC
  I just happened to stuble on this one while trying to sort out the
RISC-V bits.

gcc/ChangeLog

	* doc/tm.texi (TARGET_C_EXCESS_PRECISION): Add 16.
---
 gcc/doc/tm.texi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Palmer Dabbelt Sept. 18, 2022, 8:47 a.m. UTC | #1
On Fri, 09 Sep 2022 02:46:40 PDT (-0700), Palmer Dabbelt wrote:
> I just happened to stuble on this one while trying to sort out the
> RISC-V bits.
>
> gcc/ChangeLog
>
> 	* doc/tm.texi (TARGET_C_EXCESS_PRECISION): Add 16.
> ---
>  gcc/doc/tm.texi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
> index 858bfb80cec..7590924f2ca 100644
> --- a/gcc/doc/tm.texi
> +++ b/gcc/doc/tm.texi
> @@ -1009,7 +1009,7 @@ of the excess precision explicitly added.  For
>  @code{EXCESS_PRECISION_TYPE_FLOAT16}, and
>  @code{EXCESS_PRECISION_TYPE_FAST}, the target should return the
>  explicit excess precision that should be added depending on the
> -value set for @option{-fexcess-precision=@r{[}standard@r{|}fast@r{]}}.
> +value set for @option{-fexcess-precision=@r{[}standard@r{|}fast@r{|}16@r{]}}.
>  Note that unpredictable explicit excess precision does not make sense,
>  so a target should never return @code{FLT_EVAL_METHOD_UNPREDICTABLE}
>  when @var{type} is @code{EXCESS_PRECISION_TYPE_STANDARD},

Just pinging this one as I'm not sure if it's OK to self-approve -- no 
rush on my end, I already figured it out so I don't need the 
documentation any more.
  
Sandra Loosemore Sept. 25, 2022, 2:13 a.m. UTC | #2
On 9/18/22 02:47, Palmer Dabbelt wrote:
> On Fri, 09 Sep 2022 02:46:40 PDT (-0700), Palmer Dabbelt wrote:
>> I just happened to stuble on this one while trying to sort out the
>> RISC-V bits.
>>
>> gcc/ChangeLog
>>
>>     * doc/tm.texi (TARGET_C_EXCESS_PRECISION): Add 16.
>> ---
>>  gcc/doc/tm.texi | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
>> index 858bfb80cec..7590924f2ca 100644
>> --- a/gcc/doc/tm.texi
>> +++ b/gcc/doc/tm.texi
>> @@ -1009,7 +1009,7 @@ of the excess precision explicitly added.  For
>>  @code{EXCESS_PRECISION_TYPE_FLOAT16}, and
>>  @code{EXCESS_PRECISION_TYPE_FAST}, the target should return the
>>  explicit excess precision that should be added depending on the
>> -value set for @option{-fexcess-precision=@r{[}standard@r{|}fast@r{]}}.
>> +value set for 
>> @option{-fexcess-precision=@r{[}standard@r{|}fast@r{|}16@r{]}}.
>>  Note that unpredictable explicit excess precision does not make sense,
>>  so a target should never return @code{FLT_EVAL_METHOD_UNPREDICTABLE}
>>  when @var{type} is @code{EXCESS_PRECISION_TYPE_STANDARD},
> 
> Just pinging this one as I'm not sure if it's OK to self-approve -- no 
> rush on my end, I already figured it out so I don't need the 
> documentation any more.

This is fine, looks like a trivial correction.

-Sandra
  
Palmer Dabbelt Sept. 30, 2022, 10:25 p.m. UTC | #3
On Sat, 24 Sep 2022 19:13:36 PDT (-0700), sandra@codesourcery.com wrote:
> On 9/18/22 02:47, Palmer Dabbelt wrote:
>> On Fri, 09 Sep 2022 02:46:40 PDT (-0700), Palmer Dabbelt wrote:
>>> I just happened to stuble on this one while trying to sort out the
>>> RISC-V bits.
>>>
>>> gcc/ChangeLog
>>>
>>>     * doc/tm.texi (TARGET_C_EXCESS_PRECISION): Add 16.
>>> ---
>>>  gcc/doc/tm.texi | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
>>> index 858bfb80cec..7590924f2ca 100644
>>> --- a/gcc/doc/tm.texi
>>> +++ b/gcc/doc/tm.texi
>>> @@ -1009,7 +1009,7 @@ of the excess precision explicitly added.  For
>>>  @code{EXCESS_PRECISION_TYPE_FLOAT16}, and
>>>  @code{EXCESS_PRECISION_TYPE_FAST}, the target should return the
>>>  explicit excess precision that should be added depending on the
>>> -value set for @option{-fexcess-precision=@r{[}standard@r{|}fast@r{]}}.
>>> +value set for
>>> @option{-fexcess-precision=@r{[}standard@r{|}fast@r{|}16@r{]}}.
>>>  Note that unpredictable explicit excess precision does not make sense,
>>>  so a target should never return @code{FLT_EVAL_METHOD_UNPREDICTABLE}
>>>  when @var{type} is @code{EXCESS_PRECISION_TYPE_STANDARD},
>>
>> Just pinging this one as I'm not sure if it's OK to self-approve -- no
>> rush on my end, I already figured it out so I don't need the
>> documentation any more.
>
> This is fine, looks like a trivial correction.

Thanks, committed.
  
H.J. Lu Sept. 30, 2022, 10:51 p.m. UTC | #4
On Fri, Sep 30, 2022 at 3:25 PM Palmer Dabbelt <palmer@dabbelt.com> wrote:
>
> On Sat, 24 Sep 2022 19:13:36 PDT (-0700), sandra@codesourcery.com wrote:
> > On 9/18/22 02:47, Palmer Dabbelt wrote:
> >> On Fri, 09 Sep 2022 02:46:40 PDT (-0700), Palmer Dabbelt wrote:
> >>> I just happened to stuble on this one while trying to sort out the
> >>> RISC-V bits.
> >>>
> >>> gcc/ChangeLog
> >>>
> >>>     * doc/tm.texi (TARGET_C_EXCESS_PRECISION): Add 16.
> >>> ---
> >>>  gcc/doc/tm.texi | 2 +-
> >>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
> >>> index 858bfb80cec..7590924f2ca 100644
> >>> --- a/gcc/doc/tm.texi
> >>> +++ b/gcc/doc/tm.texi
> >>> @@ -1009,7 +1009,7 @@ of the excess precision explicitly added.  For
> >>>  @code{EXCESS_PRECISION_TYPE_FLOAT16}, and
> >>>  @code{EXCESS_PRECISION_TYPE_FAST}, the target should return the
> >>>  explicit excess precision that should be added depending on the
> >>> -value set for @option{-fexcess-precision=@r{[}standard@r{|}fast@r{]}}.
> >>> +value set for
> >>> @option{-fexcess-precision=@r{[}standard@r{|}fast@r{|}16@r{]}}.
> >>>  Note that unpredictable explicit excess precision does not make sense,
> >>>  so a target should never return @code{FLT_EVAL_METHOD_UNPREDICTABLE}
> >>>  when @var{type} is @code{EXCESS_PRECISION_TYPE_STANDARD},
> >>
> >> Just pinging this one as I'm not sure if it's OK to self-approve -- no
> >> rush on my end, I already figured it out so I don't need the
> >> documentation any more.
> >
> > This is fine, looks like a trivial correction.
>
> Thanks, committed.

tm.texi is a generated file.  I am checking in this patch to restore bootstrap.
  
Palmer Dabbelt Sept. 30, 2022, 10:58 p.m. UTC | #5
On Fri, 30 Sep 2022 15:51:02 PDT (-0700), H.J. Lu wrote:
> On Fri, Sep 30, 2022 at 3:25 PM Palmer Dabbelt <palmer@dabbelt.com> wrote:
>>
>> On Sat, 24 Sep 2022 19:13:36 PDT (-0700), sandra@codesourcery.com wrote:
>> > On 9/18/22 02:47, Palmer Dabbelt wrote:
>> >> On Fri, 09 Sep 2022 02:46:40 PDT (-0700), Palmer Dabbelt wrote:
>> >>> I just happened to stuble on this one while trying to sort out the
>> >>> RISC-V bits.
>> >>>
>> >>> gcc/ChangeLog
>> >>>
>> >>>     * doc/tm.texi (TARGET_C_EXCESS_PRECISION): Add 16.
>> >>> ---
>> >>>  gcc/doc/tm.texi | 2 +-
>> >>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> >>>
>> >>> diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
>> >>> index 858bfb80cec..7590924f2ca 100644
>> >>> --- a/gcc/doc/tm.texi
>> >>> +++ b/gcc/doc/tm.texi
>> >>> @@ -1009,7 +1009,7 @@ of the excess precision explicitly added.  For
>> >>>  @code{EXCESS_PRECISION_TYPE_FLOAT16}, and
>> >>>  @code{EXCESS_PRECISION_TYPE_FAST}, the target should return the
>> >>>  explicit excess precision that should be added depending on the
>> >>> -value set for @option{-fexcess-precision=@r{[}standard@r{|}fast@r{]}}.
>> >>> +value set for
>> >>> @option{-fexcess-precision=@r{[}standard@r{|}fast@r{|}16@r{]}}.
>> >>>  Note that unpredictable explicit excess precision does not make sense,
>> >>>  so a target should never return @code{FLT_EVAL_METHOD_UNPREDICTABLE}
>> >>>  when @var{type} is @code{EXCESS_PRECISION_TYPE_STANDARD},
>> >>
>> >> Just pinging this one as I'm not sure if it's OK to self-approve -- no
>> >> rush on my end, I already figured it out so I don't need the
>> >> documentation any more.
>> >
>> > This is fine, looks like a trivial correction.
>>
>> Thanks, committed.
>
> tm.texi is a generated file.  I am checking in this patch to restore bootstrap.

Sorry about that, and thanks for fixing it.
  

Patch

diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
index 858bfb80cec..7590924f2ca 100644
--- a/gcc/doc/tm.texi
+++ b/gcc/doc/tm.texi
@@ -1009,7 +1009,7 @@  of the excess precision explicitly added.  For
 @code{EXCESS_PRECISION_TYPE_FLOAT16}, and
 @code{EXCESS_PRECISION_TYPE_FAST}, the target should return the
 explicit excess precision that should be added depending on the
-value set for @option{-fexcess-precision=@r{[}standard@r{|}fast@r{]}}.
+value set for @option{-fexcess-precision=@r{[}standard@r{|}fast@r{|}16@r{]}}.
 Note that unpredictable explicit excess precision does not make sense,
 so a target should never return @code{FLT_EVAL_METHOD_UNPREDICTABLE}
 when @var{type} is @code{EXCESS_PRECISION_TYPE_STANDARD},