This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][AArch64][wwwdocs] Summarise some more AArch64 changes for GCC6


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]