This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1
- From: Cesar Philippidis <cesar_philippidis at mentor dot com>
- To: Thomas Schwinge <thomas at codesourcery dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 8 Jul 2014 10:01:53 -0700
- Subject: Re: [patch,gomp-4_0-branch] openacc parallel reduction part 1
- Authentication-results: sourceware.org; auth=none
- References: <53B9D780 dot 4040904 at mentor dot com> <87zjglsdc8 dot fsf at schwinge dot name> <53BC0008 dot 70404 at mentor dot com>
On 07/08/2014 07:28 AM, Cesar Philippidis wrote:
> Thanks for catching those problems! I've committed this updated version
> of the patch.
I forgot to remove the support for the collapse clause in from the loop
construct in the c frontend. I did so upstream, but not internally. I've
checked in this patch which fixes that.
Thomas, I don't know how you manage so many different branches.
Cesar
2014-07-08 Cesar Philippidis <cesar@codesourcery.com>
gcc/c/
*c-parser.c (OACC_LOOP_CLAUSE_MASK): Remove
PRAGMA_OMP_CLAUSE_COLLAPSE from theh mask.
diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c
index 66d5444..fb7e12d 100644
--- a/gcc/c/c-parser.c
+++ b/gcc/c/c-parser.c
@@ -11906,8 +11906,7 @@ c_parser_oacc_kernels (location_t loc, c_parser *parser, char *p_name)
*/
#define OACC_LOOP_CLAUSE_MASK \
- ( (OMP_CLAUSE_MASK_1 << PRAGMA_OMP_CLAUSE_COLLAPSE) \
- | (OMP_CLAUSE_MASK_1 << PRAGMA_OMP_CLAUSE_REDUCTION))
+ (OMP_CLAUSE_MASK_1 << PRAGMA_OMP_CLAUSE_REDUCTION)
static tree
c_parser_oacc_loop (location_t loc, c_parser *parser, char *p_name)