[PATCH] Documentation for loop unrolling pragmas.

Tyler Nowicki tnowicki at apple.com
Thu Jun 19 11:13:13 PDT 2014


LGTM!

On Jun 19, 2014, at 9:55 AM, Mark Heffernan <meheff at google.com> wrote:

> Thanks for the comments.  Tyler, I incorporated your suggested rewrite of the unrolling paragraph.  Hopefully it is clearer now.
> 
> I also split the vectorization/interleaving pragmas and the unrolling pragmas into separate sections.
> 
> Please take another look.
> 
> ================
> Comment at: docs/LanguageExtensions.rst:1791
> @@ +1790,3 @@
> +Loop unrolling replicates the body of a loop increasing the loop size
> +and reducing the loop count.  Loop control overhead can be reduced or
> +eliminated, and additional ILP can be exposed.  The unroller selects
> ----------------
> Eli Bendersky wrote:
>> Maybe s/loop count/loop trip count/ for clarify? There's another instance later on
> Rewrote this bit.
> 
> ================
> Comment at: docs/LanguageExtensions.rst:1793
> @@ +1792,3 @@
> +eliminated, and additional ILP can be exposed.  The unroller selects
> +an unroll count based on a limit on the growth of code size and
> +whether the loop can be unrolled completely.
> ----------------
> Eli Bendersky wrote:
>> I find this sentence a bit confusing in the context of unroll pragmas which tell the unroller exactly how much we want to unroll. OTOH this sentence seems to be describing the unrolling heuristic?
> Rewrote this bit.
> 
> http://reviews.llvm.org/D4198
> 
> 




More information about the cfe-commits mailing list