On 04/21/2016 10:21 AM, Thomas Schwinge wrote:
+ <li>Code will be offloaded onto multiple gangs, but executes with
+ just one worker, and a vector length of 1.</li>
"will be" (future) vs "executes" (present). Assuming this is all
supposed to describe current behavior, please write consistently in the
present tense.
+ Typically, using the OpenACC parallel construct will give much better
+ performance, compared to the initial support of the OpenACC kernels
+ construct.
Here too.
My only comment on the rest of the patch is that "a kernels region"
sounds like a mistake but I think that is the official terminology?
-Sandra the nit-picky