This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 3/5] OpenACC 2.0 support for libgomp - outline documentation (repost)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Julian Brown <julian at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Thomas Schwinge <thomas at codesourcery dot com>, Ilya Verbin <iverbin at gmail dot com>
- Date: Thu, 13 Nov 2014 10:40:54 +0100
- Subject: Re: [PATCH 3/5] OpenACC 2.0 support for libgomp - outline documentation (repost)
- Authentication-results: sourceware.org; auth=none
- References: <20140923192014 dot 57a2ac71 at octopus> <20141111135408 dot 281c70b6 at octopus>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Nov 11, 2014 at 01:54:08PM +0000, Julian Brown wrote:
> From 1f17beb70b5607d1884fad1cb4734857f0e7846f Mon Sep 17 00:00:00 2001
> From: Julian Brown <julian@codesourcery.com>
> Date: Mon, 22 Sep 2014 02:45:29 -0700
> Subject: [PATCH 3/5] OpenACC documentation.
>
> xxxx-xx-xx Thomas Schwinge <thomas@codesourcery.com>
> James Norris <jnorris@codesourcery.com>
>
> libgomp/
> * libgomp.texi: Outline documentation for OpenACC.
> ---
> libgomp/libgomp.texi | 661 ++++++++++++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 636 insertions(+), 25 deletions(-)
>
> diff --git a/libgomp/libgomp.texi b/libgomp/libgomp.texi
> index 254be57..9530a2b 100644
> --- a/libgomp/libgomp.texi
> +++ b/libgomp/libgomp.texi
> @@ -31,10 +31,12 @@ texts being (a) (see below), and with the Back-Cover Texts being (b)
> @ifinfo
> @dircategory GNU Libraries
> @direntry
> -* libgomp: (libgomp). GNU OpenMP runtime library
> +* libgomp: (libgomp). GNU OpenACC and OpenMP runtime library
> @end direntry
See Dave Malcolm's patch, please integrate it into your patchset.
> -This manual documents the GNU implementation of the OpenMP API for
> +This manual documents the GNU implementation of the OpenACC API for
> +offloading of code to accelerator devices in C/C++ and Fortran and
> +the GNU implementation of the OpenMP API for
> multi-platform shared-memory parallel programming in C/C++ and Fortran.
I'd write:
"This manual documents the GNU implementation of the OpenMP API
for multi-platform shared-memory parallel programming in C/C++ and Fortran
and OpenACC and OpenMP API for offloading of code to accelerator devices
from the same languages."
or so.
> +The variables @env{ACC_DEVICE_TYPE} and @env{ACC_DEVICE_NUM}
> +are defined by section 4 of the OpenACC specification in version 2.0.
> +The variable @env{GCC_ACC_NOTIFY} is used for diagnostic purposes.
> +
> +@menu
> +* ACC_DEVICE_TYPE::
> +* ACC_DEVICE_NUM::
> +* GCC_ACC_NOTIFY::
If the env var name is adjusted to GACC_NOTIFY, GOACC_NOTIFY or even
GOMP_ACC_NOTIFY (as the library is libgomp), please don't forget to change
this spot too.
> +As the OpenACC library is built using the CUDA Driver API, the question has
Is it? I thought only the nvptx plugin is.
Otherwise LGTM.
Jakub