> 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.

That's also the case for the entire subsection just above, namely "Function 
Specific Option Pragmas".  I presume the tables must be merged there too?

> 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.

Thanks, integrated into the patch.

-- 
Eric Botcazou

Reply via email to