Hi Ilmir!

On Thu, 13 Mar 2014 21:05:17 +0400, Ilmir Usmanov <i.usma...@samsung.com> wrote:
> On 07.03.2014 15:37, Ilmir Usmanov wrote:
> > I prepared simple patch to add support of OpenACC data, kernels and 
> > parallel constructs to C++ FE.
> >
> > It adds support of data clauses too.

> OK to gomp4 branch?

OK once Joseph is fine with the patch, and the following addressed:

>       * gcc/testsuite/c-c++-common/goacc/deviceptr-1.c: Move to ...
>       * gcc/testsuite/gcc.dg/goacc/deviceptr-1.c ... here.

OK, I see that the diagnostic messages do differ between C and C++, so
that it indeed seems to be better to split the file between the two.
Same for the structured block tests, sb*.*.

> --- /dev/null
> +++ b/gcc/testsuite/g++.dg/goacc-gomp/goacc-gomp.exp
> @@ -0,0 +1,38 @@
> +# Copyright (C) 2006-2013 Free Software Foundation, Inc.

2014.

> +# Main loop.
> +dg-runtest [lsort [concat \
> +     [find $srcdir/$subdir *.C] \
> +     [find $srcdir/c-c++-common/goacc-gomp *.c]]] "" "-fopenacc -fopenmp"

I see gcc/testsuite/g++.dg/gomp/gomp.exp using g++-dg-runtest instead of
plain dg-runtest.  (I have not yet looked what's different there; can you
have a look, please?)

> --- /dev/null
> +++ b/gcc/testsuite/g++.dg/goacc/goacc.exp
> @@ -0,0 +1,37 @@
> +# Copyright (C) 2006-2013 Free Software Foundation, Inc.

Likewise.

> +# Main loop.
> +dg-runtest [lsort [concat \
> +     [find $srcdir/$subdir *.C] \
> +     [find $srcdir/c-c++-common/goacc *.c]]] "" "-fopenacc"

Likewise.


Grüße,
 Thomas

Attachment: pgpfZvuNd6DyH.pgp
Description: PGP signature

Reply via email to