On 03/05/2015 04:12 PM, Mike Stump wrote:
Ping?
Just commenting on the documentation part:
Index: doc/extend.texi =================================================================== --- doc/extend.texi (revision 220084) +++ doc/extend.texi (working copy) @@ -17881,6 +17881,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 struct/union fields within structs/unions
User documentation shouldn't refer to the reader as "the programmer"; either use the second person "you" or the imperative. I'd also rearrange the paragraph slightly to put the two sentences about the parameter together, something like:
Use this pragma to inform the compiler how many times a loop should be unrolled. 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. A 0 or 1 informs the compiler to not perform any loop unrolling. -Sandra