This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Fortran OpenMP 4.0 target support

From: Tobias Burnus <burnus at net-b dot de>
To: Jakub Jelinek <jakub at redhat dot com>, gcc-patches at gcc dot gnu dot org, fortran at gcc dot gnu dot org
Date: Tue, 17 Jun 2014 23:59:22 +0200
Subject: Re: [PATCH] Fortran OpenMP 4.0 target support
Authentication-results: sourceware.org; auth=none
References: <20140617210347 dot GF31640 at tucnak dot redhat dot com>

Jakub Jelinek wrote:

This patch adds the target directives.
Tested both normally plus with target.c/splay-tree.c from
gomp-4_0-branch@203409 plus the attached patch against
target.c to implement the new to_pset map kind (5) and
allow handling of NULL.  That patch will need to be forward
ported to whatever gomp-4_0-branch now has after this is merged
from trunk to that branch.

Does this look reasonable to Fortran maintainers?

Thanks for the patch! I browsed through the patch, and it looked good tome. (However, given that the patch has "48 files changed, 3342insertions(+), 330 deletions(-)", I didn't check every line.)

If I did the book keeping correctly, a patch for an alignment test caseis still missing. As are the changes for some corner cases for which theOpenMP ARB has to provide some feedback. Any news from that side?Otherwise and aside of 4.9.1 backporting, it now looks pretty complete.


Tobias

Follow-Ups:
- Re: [PATCH] Fortran OpenMP 4.0 target support
  - From: Jakub Jelinek

References:
- [PATCH] Fortran OpenMP 4.0 target support
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]