diff mbox series

[wwwdocs] Release notes for SVE

Message ID 87k1uufmn5.fsf@linaro.org
State New
Headers show
Series [wwwdocs] Release notes for SVE | expand

Commit Message

Richard Sandiford March 2, 2018, 4:49 p.m. UTC
This patch updates the release notes to mention SVE.  OK to install?

Thanks,
Richard

Comments

Jeff Law March 2, 2018, 4:52 p.m. UTC | #1
On 03/02/2018 09:49 AM, Richard Sandiford wrote:
> This patch updates the release notes to mention SVE.  OK to install?

OK.
jeff
Gerald Pfeifer March 18, 2018, 11:05 p.m. UTC | #2
Hi Richard,

On Fri, 2 Mar 2018, Richard Sandiford wrote:
> This patch updates the release notes to mention SVE.  OK to install?


this looks fine, thank you!

> Index: wwwdocs/htdocs/gcc-8/changes.html

> ===================================================================

> +    The Scalable Vector Extension (SVE) is now supported as an

> +    optional extension to the Armv8.2-A architecture and newer.

> +    This support includes automatic vectorization with SVE instructions,


How about just saying "This includes"?

Nice job on getting this all done in time for the release, by
the way.  Chappeau!

Gerald
Richard Sandiford March 20, 2018, 8:52 a.m. UTC | #3
Gerald Pfeifer <gerald@pfeifer.com> writes:
> Hi Richard,

>

> On Fri, 2 Mar 2018, Richard Sandiford wrote:

>> This patch updates the release notes to mention SVE.  OK to install?

>

> this looks fine, thank you!

>

>> Index: wwwdocs/htdocs/gcc-8/changes.html

>> ===================================================================

>> +    The Scalable Vector Extension (SVE) is now supported as an

>> +    optional extension to the Armv8.2-A architecture and newer.

>> +    This support includes automatic vectorization with SVE instructions,

>

> How about just saying "This includes"?


I was a bit worried that that might read as "SVE includes..." rather than
"GCC's support for SVE includes...".

I thought I had to use the passive tense because all the other bullet
points in that section do, but I guess in some ways "GCC now supports..."
would avoid the shift in subject.

Thanks,
Richard
Gerald Pfeifer April 7, 2018, 9:04 p.m. UTC | #4
On Tue, 20 Mar 2018, Richard Sandiford wrote:
> I thought I had to use the passive tense because all the other bullet

> points in that section do, but I guess in some ways "GCC now supports..."

> would avoid the shift in subject.


It's a good point, actually.  Many style guides actually recommend
to avoid passive tense, and it may be worth going through the release
notes and one point and seeing what we can adjust.  In any case, it's
definitely fine to use active tense.

Gerald
diff mbox series

Patch

Index: wwwdocs/htdocs/gcc-8/changes.html
===================================================================
--- wwwdocs.orig/htdocs/gcc-8/changes.html
+++ wwwdocs/htdocs/gcc-8/changes.html
@@ -243,7 +243,18 @@  a work-in-progress.</h2>
       <li><code>+sha3</code> New SHA3 and SHA2 instructions from Armv8.4-A.  This implies <code>+sha2</code>.</li>
       <li><code>+sm4</code> New SM3 and SM4 instructions from Armv8.4-A.</li>
     </ul>
- </li>
+  </li>
+  <li>
+    The Scalable Vector Extension (SVE) is now supported as an
+    optional extension to the Armv8.2-A architecture and newer.
+    This support includes automatic vectorization with SVE instructions,
+    but it does not yet include the SVE Arm C Language Extensions (ACLE).
+    It can be enabled by specifying the <code>+sve</code> architecture
+    extension (for example, <code>-march=armv8.2-a+sve</code>).
+    By default, the generated code works with all vector lengths,
+    but it can be made specific to <i>N</i>-bit vectors using
+    <code>-msve-vector-bits=<i>N</i></code>.
+  </li>
   <li>
        Support has been added for the following processors
        (GCC identifiers in parentheses):