On 11/17/2017 03:23 AM, Eric Botcazou wrote:

Index: doc/extend.texi
===================================================================
--- doc/extend.texi     (revision 254797)
+++ doc/extend.texi     (working copy)
@@ -22376,6 +22376,18 @@ void ignore_vec_dep (int *a, int k, int
 @}
 @end smallexample
+@table @code
+@item #pragma GCC unroll @var{n}
+@cindex pragma GCC unroll @var{n}
+
+With this pragma, the programmer informs the optimizer how many times
+a loop should be unrolled.  A 0 or 1 informs the compiler to not
+perform any loop unrolling.  The pragma must be immediately before
+@samp{#pragma ivdep} or a @code{for}, @code{while} or @code{do} loop
+and applies only to the loop that follows.  @var{n} is an
+assignment-expression that evaluates to an integer constant.
+
+@end table
@node Unnamed Fields
 @section Unnamed Structure and Union Fields

This documentation patch needs some work.

First of all, the structuring in this section is screwed up. The discussion and examples for the previous item (#pragma ivdep) should be moved inside the @table so that you don't have to introduce another @table here, just insert another entry into the existing one.

Second, we shouldn't be talking about "the programmer" in the third person; programmers are "you", the readers of the manual. The paragraph structure and phrasing seem awkward as well. How about something like this instead?

You can use this pragma to control how many times a loop should be unrolled. It must be placed immediately before a @code{for}, @code{while} or @code{do} loop or a @samp{#pragma ivdep}, and applies only to the loop that follows. @var{n} is an integer constant expression; a value of 0 or 1 disables unrolling of the loop.

-Sandra

Reply via email to