Hi!

On 01/11/2016 11:35 AM, Jakub Jelinek wrote:
> On Tue, Jan 05, 2016 at 09:47:59AM -0600, James Norris wrote:
>> I've updated the original patch after some very helpful
>> comments from Sandra (thank you, thank you).
>
> I'd prefer if OpenMP
> * Enabling OpenMP::            How to enable OpenMP for your applications.
> * Runtime Library Routines::   The OpenMP runtime application programming
>                                 interface.
> * Environment Variables::      Influencing runtime behavior with environment
>                                 variables.
> chapters precede the OpenACC chapters, most libgomp users are not really
> using any offloading, which is new, but using OpenMP for host
> parallelization, and only far fewer users are actually trying some
> acceleration, whether OpenACC or OpenMP offloading parts.

OpenACC content has been moved after the OpenMP content.

>
> As Bernd found, there are some UTF-8 quotes or what in the patch, those
> need to be replaced by some texinfo markup, say
>
>> +sections 4.1 and 4.2 of the ???The OpenACC
>> +Application Programming Interface???, Version 2.0, June, 2013.}.
>
> @uref{http://www.openacc.org/, OpenACC Application Programming Interface, Version 2.0, June, 2013}
> or something similar.

Those were double quotes and have been changed to @uref's.

Patch commited to trunk

Thanks for taking time for the review.

Jim