Hi! One remark here, not related to the patch itself:
On Tue, 14 Oct 2014 17:11:18 +0100, Julian Brown <jul...@codesourcery.com>
wrote:
> --- /dev/null
> +++ b/libgomp/plugin-nvptx.c
> +//#define DISABLE_ASYNC
> +#ifndef DISABLE_ASYNC
> + [...]
> +#else
> + r = cuCtxSynchronize ();
> + if (r != CUDA_SUCCESS)
> + gomp_plugin_fatal ("cuCtxSynchronize error: %s", cuErrorMsg (r));
> +#endif
Earlier on, in discussion with Jim, we wondered whether it'd make sense
to expose the synchronous mode to the user (via an environment
variable?), which may help them when debugging? This is not a priority
right now, but perhaps something to keep in mind for later on.
Grüße,
Thomas
pgpbXGdParY9j.pgp
Description: PGP signature
