On Thu, Apr 21, 2016 at 09:15:17AM +0100, Kyrill Tkachov wrote:
> Hi all,
> 
> Here's a proposed summary of the changes in the AArch64 backend for GCC 6.
> If there's anything I've missed it's purely my oversight, feel free to add
> entries or suggest improvements.

For me, I'm mostly happy with the wording below (I've tried to be
helpful inline). But I'm not as conscientious at checking grammar as others
in the community. So this is OK from an AArch64 target perspective with
the changes below, but wait a short while to give Gerald or Sandra a chance
to comment.

> Index: htdocs/gcc-6/changes.html
> ===================================================================
> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v
> retrieving revision 1.62
> diff -U 3 -r1.62 changes.html
> --- htdocs/gcc-6/changes.html 24 Feb 2016 09:36:06 -0000      1.62
> +++ htdocs/gcc-6/changes.html 12 Apr 2016 12:47:30 -0000
> @@ -312,29 +312,91 @@
>  <h3 id="aarch64">AArch64</h3>
>     <ul>
>       <li>
> +       A number of AArch64-specific options were added.  The most important

"were" is in a different tense to the other entries in this list. To match,
you might want to use

  A number of AArch64-specific options have been added.

Or:

  A number of AArch64-specific options are now supported.

> +       ones are summarised in this section but for usage instructions please
> +       refer to the documentation.
> +     </li>
> +     <li>
>         The new command line options <code>-march=native</code>,
>         <code>-mcpu=native</code> and <code>-mtune=native</code> are now
>         available on native AArch64 GNU/Linux systems.  Specifying
>         these options will cause GCC to auto-detect the host CPU and
>         rewrite these options to the optimal setting for that system.
> -       If GCC is unable to detect the host CPU these options have no effect.
>       </li>
>       <li>
> -       <code>-fpic</code> is now supported by the AArch64 target when 
> generating
> +       <code>-fpic</code> is now supported when generating
>         code for the small code model (<code>-mcmodel=small</code>).  The 
> size of
>         the global offset table (GOT) is limited to 28KiB under the LP64 SysV 
> ABI
>         , and 15KiB under the ILP32 SysV ABI.
>       </li>
>       <li>
> -       The AArch64 port now supports target attributes and pragmas.  Please
> -       refer to the <a 
> href="https://gcc.gnu.org/onlinedocs/gcc/AArch64-Function-Attributes.html#AArch64-Function-Attributes";>
> -       documentation</a> for details of available attributes and
> +       Target attributes and pragmas are now supported.  Please
> +       refer to the documentation for details of available attributes and
>         pragmas as well as usage instructions.
>       </li>
>       <li>
>         Link-time optimization across translation units with different
>         target-specific options is now supported.
>       </li>
> +     <li>
> +       The option <code>-mtls-size=</code> is now supported.  It can be used 
> to
> +       specify the bit size of TLS offsets, allowing GCC to generate
> +       better TLS instruction sequences.
> +     </li>
> +     <li>
> +       The option <code>-fno-plt</code> is now fixed and is fully
> +       functional.

Remove "is now fixed" ?

> +     </li>
> +     <li>
> +       The ARMv8.1-A architecture and the Large System Extensions are now
> +       supported.  They can be used by specifying the
> +       <code>-march=armv8.1-a</code> option.  Additionally, the
> +       <code>+lse</code> option extension can be used in a similar fashion
> +       to other option extensions.
> +       The Large System Extensions introduce new instructions that are used
> +       in the implementation of common atomic operations.

Remove "common"

Thanks,
James

Reply via email to