This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, libgomp] Rewire OpenACC async
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Chung-Lin Tang <cltang at codesourcery dot com>
- Cc: Thomas Schwinge <thomas_schwinge at mentor dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Cesar Philippidis <cesar_philippidis at mentor dot com>, Julian Brown <julian at codesourcery dot com>
- Date: Wed, 18 May 2016 11:50:45 +0200
- Subject: Re: [PATCH, libgomp] Rewire OpenACC async
- Authentication-results: sourceware.org; auth=none
- References: <56543B8C dot 404 at codesourcery dot com> <567910D1 dot 1060306 at codesourcery dot com> <56FA4F69 dot 1060101 at codesourcery dot com> <20160512100258 dot GV28550 at tucnak dot redhat dot com> <87y47fv9hl dot fsf at hertz dot schwinge dot homeip dot net> <20160512105637 dot GA28550 at tucnak dot redhat dot com> <63d939c4-5460-cc37-6518-fe5fddeedb90 at codesourcery dot com> <20160517090059 dot GG28550 at tucnak dot redhat dot com> <8a82dd86-a4d8-28f7-8464-71a84b3c3071 at codesourcery dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Wed, May 18, 2016 at 05:40:33PM +0800, Chung-Lin Tang wrote:
> On 2016/5/17 5:00 PM, Jakub Jelinek wrote:
> > On Tue, May 17, 2016 at 04:56:42PM +0800, Chung-Lin Tang wrote:
> >> I assume that bumping GOMP_VERSION in include/gomp-constants.h would be enough?
> >
> > I think so.
> >
> > Jakub
> >
>
> How is this patch? I have added a comment to remind to adjust the version number
> when plugin interface changes are made.
Ok.
> And, if this approach to solve the plugin interface problem is okay, can I regard the
> device lock deadlock patches and async patch approved for trunk?
Yes.
> include/
> * gomp-constants.h (GOMP_VERSION): Increment to 1, add comment describe the
> need for increment this macro whenever the plugin interface is modified.
>
> Index: gomp-constants.h
> ===================================================================
> --- gomp-constants.h (revision 236363)
> +++ gomp-constants.h (working copy)
> @@ -196,8 +196,10 @@ enum gomp_map_kind
> /* Internal to libgomp. */
> #define GOMP_TARGET_FLAG_UPDATE (1U << 31)
>
> -/* Versions of libgomp and device-specific plugins. */
> -#define GOMP_VERSION 0
> +/* Versions of libgomp and device-specific plugins. GOMP_VERSION
> + should be incremented whenever an ABI-incompatible change is introduced
> + to the plugin interface defined in libgomp/libgomp.h. */
> +#define GOMP_VERSION 1
> #define GOMP_VERSION_NVIDIA_PTX 1
> #define GOMP_VERSION_INTEL_MIC 0
> #define GOMP_VERSION_HSA 0
Jakub