On 02/02/17 13:31, Gerald Pfeifer wrote:
On Thu, 2 Feb 2017, Jiong Wang wrote:
This patch adds a short entry for the -march=armv8.3-a and
-msign-return-address= options in GCC 7 to the "AArch64" section.

Thanks, Jiong.

Index: gcc-7/changes.html
===================================================================
     <li>
+       The ARMv8.3-A architecture is now supported.  It can be used by
+       specifying the <code>-march=armv8.3-a</code> option.
+
+       The option <code>-msign-return-address=</code> is supported to enable
+       return address protection using ARMv8.3-A Pointer Authentication
+       Extensions.  Please refer to the documentation for more information on
+       the arguments accepted by this option.
+     </li>

Would it make sense to make this two different items?  The way it
is currently marked up, the blank line will be "gone" once rendered.

OK, seperated them into two different items.


Where you "refer to the documentation", what kind of documentation
is that?  ARM reference manuals, GCC's documentation,...?  Being a
bit more explicit here and/or using a link would be good.

It's GCC user manual, have added the link in the updated patch.

Please review, thanks.


Index: htdocs/gcc-7/changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-7/changes.html,v
retrieving revision 1.54
diff -u -r1.54 changes.html
--- htdocs/gcc-7/changes.html	1 Feb 2017 19:23:00 -0000	1.54
+++ htdocs/gcc-7/changes.html	2 Feb 2017 14:34:49 -0000
@@ -711,6 +711,18 @@
 <h3 id="aarch64">AArch64</h3>
    <ul>
      <li>
+       The ARMv8.3-A architecture is now supported.  It can be used by
+       specifying the <code>-march=armv8.3-a</code> option.
+     </li>
+     <li>
+       The option <code>-msign-return-address=</code> is supported to enable
+       return address protection using ARMv8.3-A Pointer Authentication
+       Extensions.  For more information on the arguments accepted by this
+       option, please refer to
+	<a href="https://gcc.gnu.org/onlinedocs/gcc/AArch64-Options.html#AArch64-Options";>
+	AArch64-Options</a>.
+     </li>
+     <li>
        The ARMv8.2-A architecture and the ARMv8.2-A 16-bit Floating-Point
        Extensions are now supported.  They can be used by specifying the
        <code>-march=armv8.2-a</code> or <code>-march=armv8.2-a+fp16</code>

Reply via email to