[PATCH] D66199: [docs] loop pragmas

[Sjoerd Meijer via Phabricator via cfe-commits]

SjoerdMeijer marked an inline comment as done.
SjoerdMeijer added a comment.

> Since this is user documentation, we should only add it here once it is true.

Yep, good point, I also wanted to capture our discussions on the list. But yes, let's hold this back until we're ready with the pragmas.



================
Comment at: docs/LanguageExtensions.rst:3068-3069
 
+There are loop hints that control transformations (e.g. vectorization, loop
+unrolling) and there loop hints that set transformation options (e.g.
+``vectorize_width``, ``unroll_count``).  Pragmas setting transformation options
----------------
Meinersbur wrote:
> `vectorize_width` also "controls" a transformation. Is it that in our interpretation, `vectorize(enable)` overrides the profitability heuristic by taking out the "do not apply at all" option without setting any other option?
I think I need to think a little bit more about this:

> Is it that in our interpretation, vectorize(enable) overrides the profitability heuristic by taking out the "do not apply at all" option without setting any other option?

but I would answer this with "yes". But just checking, what exactly do you mean with "do not apply at all option"?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66199/new/

https://reviews.llvm.org/D66199