[prev in list] [next in list] [prev in thread] [next in thread]
List: cfe-commits
Subject: Re: [PATCH] D11182: [OPENMP 4.0] Initial support for 'omp declare reduction' construct.
From: "Bataev, Alexey via cfe-commits" <cfe-commits () lists ! llvm ! org>
Date: 2015-08-21 3:20:23
Message-ID: BLU436-SMTP253301F82AA56EE3A84F59B85650 () phx ! gbl
[Download RAW message or body]
I'll update patch, Michael
Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team
20.08.2015 18:40, Michael Wong пишет:
> fraggamuffin added a comment.
>
>
>
>
> ================
> Comment at: lib/Parse/ParseDeclCXX.cpp:3011
> @@ -3010,3 +3010,3 @@
> if (Tok.is(tok::annot_pragma_openmp)) {
> - ParseOpenMPDeclarativeDirective();
> continue;
> ----------------
> While testing this patch with the latest trunk for my work on declare target, I \
> kept getting a build error in ParseDeclCXX.cpp as there seems to be no CurAS in \
> scope, this may be because of recent changes. But this line: if \
> (Tok.is(tok::annot_pragma_openmp)) return ParseOpenMPDeclarativeDirective(CurAS);
> Seems to work better as
> if (Tok.is(tok::annot_pragma_openmp))
> return ParseOpenMPDeclarativeDirective(AS);
>
>
> http://reviews.llvm.org/D11182
>
>
>
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic