This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [2/2] OpenACC routine support
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Nathan Sidwell <nathan at acm dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 3 Nov 2015 17:04:51 +0100
- Subject: Re: [2/2] OpenACC routine support
- Authentication-results: sourceware.org; auth=none
- References: <5637B1CF dot 5060408 at acm dot org> <5637B827 dot 1020909 at acm dot org> <20151103153844 dot GR478 at tucnak dot redhat dot com> <5638D935 dot 3020808 at acm dot org>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Nov 03, 2015 at 10:56:37AM -0500, Nathan Sidwell wrote:
> On 11/03/15 10:38, Jakub Jelinek wrote:
> >On Mon, Nov 02, 2015 at 02:23:19PM -0500, Nathan Sidwell wrote:
> >>Here are the tests for the routine support. The compiler tests check
> >>invalid combinations of gang, worker, vector & seq. The libgomp execution
> >>tests check the expected partioning occurs within loops. As with the
> >>reduction tests, these ones are taken from the execution model loop tests.
> >
> >I find the testsuite coverage insufficient, e.g. you don't have equivalent
> >of first half of declare-simd-2.C or declare-simd-2.c
> >(everything above #pragma omp declare simd inbranch notinbranch),
> >to verify that if acc routine is used without the (fnname) in it, then
> >it can't be followed by var definition and various other tokens.
>
> d'oh! forgot to port those tests. Easy fix.
>
> ok with that added?
Yes.
Jakub