Re: [patch, avr,wwwdocs] PR 58655

[Pitchumani Sivanupandi <pitchumani.sivanupandi@atmel.com>]

[-- Attachment #1: Type: text/plain, Size: 2129 bytes --]

On Tuesday 21 June 2016 09:39 PM, Georg-Johann Lay wrote:
> Pitchumani Sivanupandi schrieb:
>> Attached patches add documentation for -mfract-convert-truncate option
>> and add that info to release notes (gcc-4.9 changes).
>>
>> If OK, could someone commit please? I do not have commit access.
>>
>> Regards,
>> Pitchumani
>>
>> gcc/ChangeLog
>>
>> 2016-06-21  Pitchumani Sivanupandi <pitchumani.s@atmel.com>
>>
>>     PR target/58655
>>     * doc/invoke.texi (AVR Options): Document -mfract-convert-truncate
>>     option.
>>
>> --- a/wwwdocs/htdocs/gcc-4.9/changes.html
>> +++ b/wwwdocs/htdocs/gcc-4.9/changes.html
>> @@ -579,6 +579,14 @@ auto incr(T x) { return x++; }
>>         size when compiling for the M-profile processors.
>>       </li>
>>       </ul>
>> +<h3 id="avr">AVR</h3>
>> +<ul>
>> +  <li>
>> +    A new command-line option -mfract-convert-truncate has been added.
>
> <code> tags around the option.
>
>> +    It allows compiler to use truncation instead of rounding towards
>> +    0 for fractional int types.
>
> "zero" instead of "0", and it's for fixed-point types, not for int types.
>
>> +  </li>
>> +</ul>
>>  <h3 id="x86">IA-32/x86-64</h3>
>>    <ul>
>>      <li><code>-mfpmath=sse</code> is now implied by
>> <code>-ffast-math</code>
...
>>  @emph{Blackfin Options}
>>  @gccoptlist{-mcpu=@var{cpu}@r{[}-@var{sirevision}@r{]} @gol
>> @@ -14586,6 +14586,10 @@ sbiw r26, const   ; X -= const
>>  @opindex mtiny-stack
>>  Only change the lower 8@tie{}bits of the stack pointer.
>>
>> +@item -mfract-convert-truncate
>> +@opindex mfract-convert-truncate
>> +Allow to use truncation instead of rounding towards 0 for fractional
>> int types.
>
> Same here: "zero" and "fixed-point".
>
>> +
>>  @item -nodevicelib
>>  @opindex nodevicelib
>>  Don't link against AVR-LibC's device specific library
>> @code{lib<mcu>.a}.
>
Thanks Johann.

Updated the patches.

Regards,
Pitchumani

gcc/ChangeLog

2016-06-22  Pitchumani Sivanupandi  <pitchumani.s@atmel.com>

      PR target/58655
      * config/avr/avr.opt (-mfract-convert-truncate): Update description.
      * doc/invoke.texi (AVR Options): Document it.



[-- Attachment #2: PR58655-doc.patch --]
[-- Type: text/x-patch, Size: 1621 bytes --]

diff --git a/gcc/config/avr/avr.opt b/gcc/config/avr/avr.opt
index 05aa4b6..1af792b 100644
--- a/gcc/config/avr/avr.opt
+++ b/gcc/config/avr/avr.opt
@@ -97,7 +97,7 @@ Warn if the ISR is misspelled, i.e. without __vector prefix. Enabled by default.
 
 mfract-convert-truncate
 Target Report Mask(FRACT_CONV_TRUNC)
-Allow to use truncation instead of rounding towards 0 for fractional int types.
+Allow to use truncation instead of rounding towards zero for fractional fixed-point types.
 
 nodevicelib
 Driver Target Report RejectNegative
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index e000218..040fb6e 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -643,8 +643,8 @@ Objective-C and Objective-C++ Dialects}.
 @emph{AVR Options}
 @gccoptlist{-mmcu=@var{mcu} -maccumulate-args -mbranch-cost=@var{cost} @gol
 -mcall-prologues -mint8 -mn_flash=@var{size} -mno-interrupts @gol
--mrelax -mrmw -mstrict-X -mtiny-stack -nodevicelib -Waddr-space-convert @gol
--Wmisspelled-isr}
+-mrelax -mrmw -mstrict-X -mtiny-stack -mfract-convert-truncate -nodevicelib @gol
+-Waddr-space-convert -Wmisspelled-isr}
 
 @emph{Blackfin Options}
 @gccoptlist{-mcpu=@var{cpu}@r{[}-@var{sirevision}@r{]} @gol
@@ -14586,6 +14586,10 @@ sbiw r26, const   ; X -= const
 @opindex mtiny-stack
 Only change the lower 8@tie{}bits of the stack pointer.
 
+@item -mfract-convert-truncate
+@opindex mfract-convert-truncate
+Allow to use truncation instead of rounding towards zero for fractional fixed-point types.
+
 @item -nodevicelib
 @opindex nodevicelib
 Don't link against AVR-LibC's device specific library @code{lib<mcu>.a}.


[-- Attachment #3: PR58655-gcc-49-changes.patch --]
[-- Type: text/x-patch, Size: 589 bytes --]

--- a/wwwdocs/htdocs/gcc-4.9/changes.html
+++ b/wwwdocs/htdocs/gcc-4.9/changes.html
@@ -579,6 +579,14 @@ auto incr(T x) { return x++; }
        size when compiling for the M-profile processors.
      </li>
      </ul>
+<h3 id="avr">AVR</h3>
+<ul>
+  <li>
+    A new command-line option <code>-mfract-convert-truncate</code> has been
+    added. It allows compiler to use truncation instead of rounding towards
+    zero for fractional fixed-point types.
+  </li>
+</ul>
 <h3 id="x86">IA-32/x86-64</h3>
   <ul>
     <li><code>-mfpmath=sse</code> is now implied by <code>-ffast-math</code>