[wwwdocs] Mention xgene-1 in arm and aarch64, FreeBSD support for arm

Hi all,

This patch to changes.html mentions the xgene1 support in GCC 5 for arm 
and aarch64 and also the FreeBSD support for ARM.

Is this ok?
I added the FreeBSD part in the Operating Systems section similar to 
Dragonfly BSD.

Thanks,
Kyrill

On 13/02/15 09:52, Kyrill Tkachov wrote:
> Hi all,
> 
> This patch to changes.html mentions the xgene1 support in GCC 5 for arm
> and aarch64 and also the FreeBSD support for ARM.
> 
> Is this ok?

The repetitive nature of all these new cpus being added looks rather
wooden.  I think it would be better to merge them into one change block,
that lists all the cpus and their internal names, then mentions once at
the end that these names can be used as arguments to -mcpu and -mtune.

R.

> I added the FreeBSD part in the Operating Systems section similar to
> Dragonfly BSD.
> 
> Thanks,
> Kyrill
> 
> wwwdocs-xgene-freebsd.patch
> 
> 
> Index: htdocs/gcc-5/changes.html
> ===================================================================
> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v
> retrieving revision 1.80
> diff -U 3 -r1.80 changes.html
> --- htdocs/gcc-5/changes.html	9 Feb 2015 11:54:27 -0000	1.80
> +++ htdocs/gcc-5/changes.html	11 Feb 2015 17:24:14 -0000
> @@ -480,6 +480,9 @@
>       <li>Support for the Cavium ThunderX processor is now available through the
>      <code>-mcpu=thunderx</code> and <code>-mtune=thunderx</code> options.
>       </li>
> +     <li>Support for the Applied Micro X-Gene processor is now available through
> +     the <code>-mcpu=xgene1</code> and <code>-mtune=xgene1</code> options.
> +     </li>
>       <li>Support for the Cortex-A72 processor has been added through
>         the <code>-mcpu=cortex-a72</code> option and the big.LITTLE
>         variant <code>-mcpu=cortex-a72.cortex-a53</code>.  Using these options
> @@ -512,6 +515,9 @@
>        <li> Support for the Cortex-A17 processor has been added through the
>        <code>-mcpu=cortex-a17</code> and <code>-mtune=cortex-a17</code> options.
>        </li>
> +     <li>Support for the Applied Micro X-Gene processor is now available through
> +     the <code>-mcpu=xgene1</code> and <code>-mtune=xgene1</code> options.
> +     </li>
>        <li> Support for the Cortex-M7 processor has been added through the
>        <code>-mcpu=cortex-m7</code> and <code>-mtune=cortex-m7</code> options.
>        </li>
> @@ -666,6 +672,13 @@
>      <li>GCC now supports the DragonFly BSD operating system.</li>
>    </ul>
>  
> +  <h3 id="freebsd">FreeBSD</h3>
> +
> +  <ul>
> +    <li>GCC now supports the FreeBSD operating system for the arm port
> +    through the <code>arm*-*-freebsd*</code> target triplets.</li>
> +  </ul>
> +
>    <h3 id="vxmils">VxWorks MILS</h3>
>  
>    <ul>
>

> On 13 Feb 2015, at 11:14, Richard Earnshaw <Richard.Earnshaw@foss.arm.com> wrote:
> 
>> Is this ok?
> 
> The repetitive nature of all these new cpus being added looks rather
> wooden.  I think it would be better to merge them into one change block,
> that lists all the cpus and their internal names, then mentions once at
> the end that these names can be used as arguments to -mcpu and -mtune.

I wholeheartedly agree with Richard on this.

>> +     <li>Support for the Applied Micro X-Gene processor is now available through
>> +     the <code>-mcpu=xgene1</code> and <code>-mtune=xgene1</code> options.
>> +     </li>

Please change this to “X-Gene 1 processor”, as there will be a separate option for “X-Gene 2”
in the future (which will switch to slightly different tuning parameters).

Best,
Philipp.